Max/Backend/Curriculum #106

Merged
tonitch merged 6 commits from Max/Backend/Curriculum into master 2024-03-15 22:33:25 +01:00
Owner
No description provided.
Maxime added 5 commits 2024-03-14 21:56:07 +01:00
added InscriptionController
All checks were successful
Build and test backend / Build-backend (pull_request) Successful in 2m2s
Build and test backend / Test-backend (pull_request) Successful in 1m59s
Build and test FrontEnd / Build-frontend (pull_request) Successful in 24s
f0a411c031
added cursus/course interactions
All checks were successful
Build and test backend / Build-backend (pull_request) Successful in 2m0s
Build and test backend / Test-backend (pull_request) Successful in 2m2s
Build and test FrontEnd / Build-frontend (pull_request) Successful in 25s
12846ed83d
tonitch requested changes 2024-03-15 08:58:47 +01:00
tonitch left a comment
Owner

Sauf pour le /curriculums le reste n'est pas "bloquant" et c'est lgtm pour moi

Sauf pour le /curriculum**s** 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);
}
Owner

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 ?

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 ?
Maxime marked this conversation as resolved
@ -0,0 +33,4 @@
public Iterable<Cursus> findAll(){
return cursusRepo.findAll();
}
}
Owner

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.

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.
Maxime marked this conversation as resolved
@ -20,0 +14,4 @@
}
public void save(InscriptionRequest inscriptionRequest){
inscriptionRepo.save(inscriptionRequest);
Owner

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)

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)
Maxime marked this conversation as resolved
@ -26,3 +21,1 @@
return null;
}
return inscriptionRequest;
return inscriptionRepo.findById(id);
Owner

lol

lol
Maxime marked this conversation as resolved
Maxime added 2 commits 2024-03-15 17:30:06 +01:00
to better suit the backend and logic (register => list and register => single item)
fixed inscription in the backend
All checks were successful
Build and test backend / Build-backend (pull_request) Successful in 2m8s
Build and test backend / Test-backend (pull_request) Successful in 2m0s
Build and test FrontEnd / Build-frontend (pull_request) Successful in 24s
017235cccf
Maxime added 1 commit 2024-03-15 17:41:31 +01:00
Merge branch 'master' into Max/Backend/Curriculum
Some checks failed
Build and test backend / Build-backend (pull_request) Successful in 2m3s
Build and test backend / Test-backend (pull_request) Successful in 2m0s
Build and test FrontEnd / Build-frontend (pull_request) Failing after 21s
f33a421aee
Maxime requested review from tonitch 2024-03-15 17:42:59 +01:00
Maxime requested review from LeoMoulin 2024-03-15 17:43:06 +01:00
Maxime requested review from Wal 2024-03-15 17:43:06 +01:00
Maxime added the
Done
back
URGENT BORDEL DE Q
labels 2024-03-15 17:43:20 +01:00
LeoMoulin approved these changes 2024-03-15 17:59:47 +01:00
Wal requested changes 2024-03-15 18:34:07 +01:00
Wal left a comment
Owner

Modifie juste le crochet qui nik le build dans serviceinscription.js et ca me semble ok

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"});
Owner

un peu weird le crochet fermant sans ouverture

un peu weird le crochet fermant sans ouverture
Author
Owner

écoute ct déjà là avant mais je vais retirer ça de suite

écoute ct déjà là avant mais je vais retirer ça de suite
Owner

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

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
Maxime added 1 commit 2024-03-15 18:54:34 +01:00
removed missclick
All checks were successful
Build and test backend / Build-backend (pull_request) Successful in 2m6s
Build and test backend / Test-backend (pull_request) Successful in 2m2s
Build and test FrontEnd / Build-frontend (pull_request) Successful in 23s
2ea6ed16bb
Wal approved these changes 2024-03-15 18:54:59 +01:00
tonitch approved these changes 2024-03-15 21:55:42 +01:00
tonitch merged commit 9f02207eb0 into master 2024-03-15 22:33:25 +01:00
tonitch deleted branch Max/Backend/Curriculum 2024-03-15 22:33:25 +01:00
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
4 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: PGL/Clyde#106
No description provided.