Skip to content

Commit 0c6c83e

Browse files
committed
fix: do not add classroom when name already exists
closes #300
1 parent e403ce8 commit 0c6c83e

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

public/app/listeners/addClassroom.js

+3-3
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ const addClassroom = (mainWindow, db) => async (event, { name, userId }) => {
3232
.find({ name, teacherId: userId })
3333
.value();
3434
if (found) {
35-
mainWindow.webContents.send(
35+
return mainWindow.webContents.send(
3636
ADD_CLASSROOM_CHANNEL,
3737
ERROR_DUPLICATE_CLASSROOM_NAME
3838
);
@@ -58,10 +58,10 @@ const addClassroom = (mainWindow, db) => async (event, { name, userId }) => {
5858
.push(newClassroom)
5959
.write();
6060

61-
mainWindow.webContents.send(ADD_CLASSROOM_CHANNEL, newClassroom);
61+
return mainWindow.webContents.send(ADD_CLASSROOM_CHANNEL, newClassroom);
6262
} catch (err) {
6363
logger.error(err);
64-
mainWindow.webContents.send(ADD_CLASSROOM_CHANNEL, ERROR_GENERAL);
64+
return mainWindow.webContents.send(ADD_CLASSROOM_CHANNEL, ERROR_GENERAL);
6565
}
6666
};
6767

0 commit comments

Comments
 (0)