[GH-ISSUE #41] Add update controllers #16

Closed
opened 2026-09-04 00:21:36 +02:00 by Stone_Red · 4 comments
Owner

Originally created by @stephan418 on GitHub (May 1, 2022).
Original GitHub issue: https://github.com/detleph/server/issues/41

Originally assigned to: @Flexla54 on GitHub.

It is currently not possible to update most of the resources. This is functionality that should be extended (Maybe @Flexla54 for experimenting with express)

Originally created by @stephan418 on GitHub (May 1, 2022). Original GitHub issue: https://github.com/detleph/server/issues/41 Originally assigned to: @Flexla54 on GitHub. It is currently not possible to update most of the resources. This is functionality that should be extended (Maybe @Flexla54 for experimenting with express)
Author
Owner

@stephan418 commented on GitHub (May 3, 2022):

This could be done using PATCH for partial updates and PUT for complete writeovers

<!-- gh-comment-id:1115909896 --> @stephan418 commented on GitHub (May 3, 2022): This could be done using PATCH for partial updates and PUT for complete writeovers
Author
Owner

@Flexla54 commented on GitHub (May 29, 2022):

I think all controllers own one or more update methods. Please check feature-roles branch if all update methods exist.
Note that updateDiscipline is implemented twice (one in feature-roles, the other in feature-endpoints) - when merging use feature-endpoints!

<!-- gh-comment-id:1140355833 --> @Flexla54 commented on GitHub (May 29, 2022): I think all controllers own one or more update methods. Please check feature-roles branch if all update methods exist. Note that updateDiscipline is implemented twice (one in feature-roles, the other in feature-endpoints) - when merging use feature-endpoints!
Author
Owner

@stephan418 commented on GitHub (May 29, 2022):

Impressive. Will review the code and get back

<!-- gh-comment-id:1140398357 --> @stephan418 commented on GitHub (May 29, 2022): Impressive. Will review the code and get back
Author
Owner

@stephan418 commented on GitHub (Jun 3, 2022):

Seems mostly finished.

<!-- gh-comment-id:1145825745 --> @stephan418 commented on GitHub (Jun 3, 2022): Seems mostly finished.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: detleph/server#16