- Belgium
- http://tcaresset.com
-
admin of herisson
- Joined on
2022-03-15
Pour le coup je sais que par convention il faudrait les mettre en majuscule parce que c'est des constantes mais vu que je le récupère au niveau backend pour charger les fichiers est-ce qu'il serait possible de faire les noms en PascalCase ?
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 ^^
Comme je l'ai dit directement hier, c'est dommage parce que sa obfusque la fonction alors que tu pourrais faire directement inscriptionService.inscriptionRepo.save(inscriptionRequest) qui te retournerais en plus ton repo. Mais dans les faits ça n'est pas génant pour le projet non plus so you do you... hopefully ça ne vas pas géner quand on fera nos extensions individuelles x)
Sauf pour le /curriculums le reste n'est pas "bloquant" et c'est lgtm pour moi
Si je comprends bien, la différence entre GET /curriculums et GET /curriculum c'est que l'un retourne la liste des cours avec et l'autre non. Je trouve ça un peu abstrait, peut être est-il possible de faire un nom plus descriptif ?