Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

POST on contained resource should not update its container #7

Closed
CxRes opened this issue May 30, 2024 · 0 comments · Fixed by #8
Closed

POST on contained resource should not update its container #7

CxRes opened this issue May 30, 2024 · 0 comments · Fixed by #8

Comments

@CxRes
Copy link
Member

CxRes commented May 30, 2024

Triggering an update on a container, when POST is used on a contained resource is incorrect, as the POST is modifying the contained resource and create a resource still inside that.
CxRes added a commit to CxRes/solid-prep that referenced this issue May 30, 2024
@CxRes
fix: Unnecessary Update on Parent Container on POST
328db28 
POST should not trigger an update on the parent container of the resource. This case is therefore removed from the non-normative requirements under Activity Mapping.

Fixes solid#7
@CxRes CxRes mentioned this issue May 30, 2024
Merged
@CxRes CxRes closed this as completed in #8 May 30, 2024
@CxRes CxRes closed this as completed in 4a3e2f6 May 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: Unnecessary Update on Parent Container on POST CxRes/solid-prep
1 participant
@CxRes