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

PMPR-36 : PUT and DELETE /ues/{idUE} #124

Merged
djalim merged 1 commits from feature/delete-ues-id-36 into develop 2026-04-23 11:47:48 +00:00
Conversation 5 Commits 1 Files Changed 1
+67 -1
clement commented 2026-04-22 16:40:11 +00:00
Collaborator
Copy Link
Copy Source

Ajout de la route DELETE /ues/{idUE} pour supprimer une UE

Ajout de la route DELETE /ues/{idUE} pour supprimer une UE
anys requested review from anys 2026-04-22 16:48:43 +00:00
anys commented 2026-04-22 16:51:06 +00:00
Collaborator
Copy Link
Copy Source

The DELETE route seems correct, but this branch also includes GET and PUT routes.
This is similar to #121

The DELETE route seems correct, but this branch also includes GET and PUT routes. This is similar to #121
anys commented 2026-04-22 17:02:31 +00:00
Collaborator
Copy Link
Copy Source

Instead of having 3 different pull request, maybe close #120 and #121 and rename this one to reference the GET and PUT request ?

Instead of having 3 different pull request, maybe close #120 and #121 and rename this one to reference the GET and PUT request ?
anys declined to review 2026-04-22 17:14:36 +00:00
anys requested review from djalim 2026-04-22 17:14:38 +00:00
djalim commented 2026-04-22 17:25:17 +00:00
Collaborator
Copy Link
Copy Source

same as here

same as [here](https://git.polytech.djalim.fr/admin/PolyMPR/pulls/119#issuecomment-783)
djalim changed title from PMPR-36 : DELETE /ues/{idUE} - supprimer une UE to PMPR-36 : PUT and DELETE /ues/{idUE} 2026-04-23 11:41:39 +00:00
djalim commented 2026-04-23 11:42:37 +00:00
Collaborator
Copy Link
Copy Source

as the conflict has not been fully fixed, the put handler is still on this branch, #121 has been closed and this will be merged

as the conflict has not been fully fixed, the put handler is still on this branch, #121 has been closed and this will be merged
djalim added 1 commit 2026-04-23 11:44:58 +00:00
PMPR-36 : DELETE /ues/{idUE} - supprimer une UE eeb087ea76
djalim force-pushed feature/delete-ues-id-36 from c405eca4f3 to eeb087ea76 2026-04-23 11:44:58 +00:00 Compare
djalim commented 2026-04-23 11:47:44 +00:00
Collaborator
Copy Link
Copy Source

closes #35 and #36

closes #35 and #36
djalim merged commit eeb087ea76 into develop 2026-04-23 11:47:48 +00:00
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#124
Delete Branch "feature/delete-ues-id-36"

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?