admin/PolyMPR
1
0
Fork 0
Code Issues 14 Pull Requests 1 Actions Packages Projects 1 Releases Wiki Activity

PMPR-38 : POST /ue-modules - associer un module à une UE #119

Merged
djalim merged 1 commits from feature/post-ue-modules-38 into develop 2026-04-22 23:02:53 +00:00
Conversation 4 Commits 1 Files Changed 1
+22
clement commented 2026-04-22 16:13:14 +00:00
Collaborator
Copy Link
Copy Source

Ajout de la route POST /ue-modules pour associer un module à une UE

Ajout de la route POST /ue-modules pour associer un module à une UE
anys commented 2026-04-22 17:06:10 +00:00
Collaborator
Copy Link
Copy Source

Seems good but :

  • You don't seem to verify the type of the const
  • This branch also contains the GET request. Maybe close #118 ?
Seems good but : - You don't seem to verify the type of the const - This branch also contains the GET request. Maybe close #118 ?
anys requested review from anys 2026-04-22 17:07:16 +00:00
anys declined to review 2026-04-22 17:14:50 +00:00
anys requested review from djalim 2026-04-22 17:14:52 +00:00
djalim commented 2026-04-22 17:19:59 +00:00
Collaborator
Copy Link
Copy Source

The GET handler was merged in a previous branch, clean this one by making a final commit without the GET handler or push ammend and push force without the staged handler

The GET handler was merged in a previous branch, clean this one by making a final commit without the GET handler or push ammend and push force without the staged handler
clement added 1 commit 2026-04-22 18:45:31 +00:00
PMPR-38 : POST /ue-modules - associer un module à une UE 79669d60cf
clement force-pushed feature/post-ue-modules-38 from 60c8a6f3cf to b6ad667e90 2026-04-22 18:45:31 +00:00 Compare
djalim force-pushed feature/post-ue-modules-38 from b6ad667e90 to 79669d60cf 2026-04-22 23:00:10 +00:00 Compare
djalim commented 2026-04-22 23:02:43 +00:00
Collaborator
Copy Link
Copy Source

cleaned up branch, lgtm

cleaned up branch, lgtm
djalim merged commit 79669d60cf into develop 2026-04-22 23:02:53 +00:00
djalim commented 2026-04-22 23:03:10 +00:00
Collaborator
Copy Link
Copy Source

closes #38

closes #38
Sign in to join this conversation.
Reviewers
No Reviewers
djalim
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
anys clement djalim ibra admin
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: admin/PolyMPR#119
Delete Branch "feature/post-ue-modules-38"

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?