PGL/Clyde
PGL/Clyde
3
2
Fork 3
Code Issues 13 Pull Requests Actions Projects 3 Releases 2 Wiki Activity

added ApplicationController and Application enum #119

Merged
tonitch merged 4 commits from Max/Backend/AppsController into master 2024-03-16 14:51:57 +01:00
Conversation 0 Commits 4 Files Changed 1 +6 -6
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
backend/src/main/java/ovh/herisson/Clyde/EndPoints/ApplicationsController.java
View File

@ -46,11 +46,12 @@ public class ApplicationsController {
authorizedApps.add(Applications.Login);
authorizedApps.add(Applications.Profile);
if (posterRole == Role.Teacher || posterRole == Role.Student || posterRole == Role.Admin){
authorizedApps.add(Applications.Msg);
authorizedApps.add(Applications.Forum);
authorizedApps.add(Applications.Rdv);
Maxime marked this conversation as resolved Outdated
tonitch commented 2024-03-15 22:20:03 +01:00
Outdated
Review
Copy Link

cette ligne est un peut bizzare dans le sens ou si il est admin il va retourner direct et par récupérer les autres... Aussi de base msg forum et rdv c'est que pour les student et profs. Si c'est chiant pourquoi pas laisser msg et forum mais rdv c'est spécifique aux profs et etudiants ^^

cette ligne est un peut bizzare dans le sens ou si il est admin il va retourner direct et par récupérer les autres... Aussi de base msg forum et rdv c'est que pour les student et profs. Si c'est chiant pourquoi pas laisser msg et forum mais rdv c'est spécifique aux profs et etudiants ^^
Maxime commented 2024-03-15 23:57:33 +01:00
Outdated
Review
Copy Link

petit mistake oui bien vu !

petit mistake oui bien vu !
if (posterRole == Role.Student || posterRole == Role.Admin) return authorizedApps;
}
if (posterRole == Role.Teacher || posterRole == Role.Secretary || posterRole == Role.Admin) authorizedApps.add(Applications.ManageCourses);
@ -58,5 +59,4 @@ public class ApplicationsController {
return authorizedApps;
}
}