- Belgium
- http://tcaresset.com
-
admin of herisson
- Joined on
2022-03-15
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 ?
Sauf pour le /curriculums le reste n'est pas "bloquant" et c'est lgtm pour moi
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)
juste pour l'example du commentaire suivants, ce service complet pourrait est dispensable puisqu'il ne fait que délivrer des fonctions déjà existante. Mais je comprends que pour la structure du projet cette class existe so is ok.
askip en java une meilleur approche serait d'utiliser les EnumSet https://stackoverflow.com/questions/6067411/checking-flag-bits-java/6067501#6067501
Ca semble