Skip to content

Commit 3080cca

Browse files
committed
#3936 service returns nothing, remove caps for route
1 parent 536fea3 commit 3080cca

2 files changed

Lines changed: 2 additions & 4 deletions

File tree

src/backend/src/routes/recruitment.routes.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,6 @@ recruitmentRouter.post(
6464
RecruitmentController.createGuestDefinition
6565
);
6666

67-
recruitmentRouter.delete('/guestDefinition/:definitionId/delete', RecruitmentController.deleteGuestDefinition);
67+
recruitmentRouter.delete('/guestdefinition/:definitionId/delete', RecruitmentController.deleteGuestDefinition);
6868

6969
export default recruitmentRouter;

src/backend/src/services/recruitment.services.ts

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -259,7 +259,7 @@ export default class RecruitmentServices {
259259
* @param deleter the user requesting to delete the guestDefinition
260260
* @param organizationId the organization ID of the deleter
261261
*/
262-
static async deleteGuestDefinition(deleter: User, definitionId: string, organization: Organization) {
262+
static async deleteGuestDefinition(deleter: User, definitionId: string, organization: Organization): Promise<void> {
263263
if (!(await userHasPermission(deleter.userId, organization.organizationId, isAdmin))) {
264264
throw new AccessDeniedAdminOnlyException('delete a guestDefinition');
265265
}
@@ -273,7 +273,5 @@ export default class RecruitmentServices {
273273
where: { definitionId },
274274
data: { dateDeleted: new Date(), userDeletedId: deleter.userId }
275275
});
276-
277-
return def;
278276
}
279277
}

0 commit comments

Comments
 (0)