Max/Backend/Curriculum #106
Labels
No Label
Bug
Done
Pas urgent
Proposition
Question
TODO
Tests
URGENT BORDEL DE Q
Waiting for review
back
front
No Milestone
No project
No Assignees
4 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: PGL/Clyde#106
Loading…
Reference in New Issue
Block a user
No description provided.
Delete Branch "Max/Backend/Curriculum"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Sauf pour le /curriculums le reste n'est pas "bloquant" et c'est lgtm pour moi
@ -0,0 +43,4 @@
@GetMapping("/curriculum")
public ResponseEntity<Iterable<CursusCourse>> findAll(){
return new ResponseEntity<>(cursusCourseServ.findAll(),HttpStatus.OK);
}
Si je comprends bien, la différence entre
GET /curriculums
etGET /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 ?@ -0,0 +33,4 @@
public Iterable<Cursus> findAll(){
return cursusRepo.findAll();
}
}
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.
@ -20,0 +14,4 @@
}
public void save(InscriptionRequest inscriptionRequest){
inscriptionRepo.save(inscriptionRequest);
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)@ -26,3 +21,1 @@
return null;
}
return inscriptionRequest;
return inscriptionRepo.findById(id);
lol
Modifie juste le crochet qui nik le build dans serviceinscription.js et ca me semble ok
@ -11,3 +11,3 @@
*/
export async function createRegister(){
return restPost("/requests/register");
return restPost("/request/register"});
un peu weird le crochet fermant sans ouverture
écoute ct déjà là avant mais je vais retirer ça de suite
Tu me demandes de review je le fait la si on avait laisser passer ca m'aurait niqué pour le front si j'utilisais la function alors mieux vaut le faire mtn