Merge pull request #509 from bluewave-labs/feat/be/edit-monitor-notifications

Notifications removed and readded on monitor edit
This commit is contained in:
Alexander Holliday authored and GitHub committed 2024-07-31 14:23:41 -07:00
commit baa3a1169d
2 files changed
+18

No files matched your search

+16
View File
@@ -234,7 +234,23 @@ const editMonitor = async (req, res, next) => {
try {
const monitorBeforeEdit = await req.db.getMonitorById(req, res);
// Get notifications from the request body
const notifications = req.body.notifications;
const editedMonitor = await req.db.editMonitor(req, res);
await req.db.deleteNotificationsByMonitorId(editedMonitor._id);
if (notifications && notifications.length !== 0) {
await Promise.all(
notifications.map(async (notification) => {
notification.monitorId = editedMonitor._id;
await req.db.createNotification(notification);
})
);
}
// Delete the old job(editedMonitor has the same ID as the old monitor)
await req.jobQueue.deleteJob(monitorBeforeEdit);
// Add the new job back to the queue
+2
View File
@@ -209,6 +209,8 @@ const deleteMonitorsByUserId = async (userId) => {
const editMonitor = async (req, res) => {
const candidateId = req.params.monitorId;
const candidateMonitor = req.body;
candidateMonitor.notifications = undefined;
try {
const editedMonitor = await Monitor.findByIdAndUpdate(
candidateId,