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.
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
HTTP API fixes #156
HTTP API fixes #156
Conversation
@@ -301,7 +302,6 @@ public Response updateSparql(@PathParam("path") | |||
} | |||
|
|||
session.save(); | |||
versionService.checkpoint(session, path); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it intentional to remove all versioning calls from FedoraNodes?
Here, and at lines 375, 512, 605, 646.
@mikedurbin, thoughts?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a bad rebase, I think.
created resource; fixes #60887246
…he location header. Fixes #60886000
fixes #60884508
Fixes #60875916
fixes #60877478
Fixed up. |
HTTP API fixes Resolves: https://www.pivotaltracker.com/story/show/60887336 https://www.pivotaltracker.com/story/show/60887246 https://www.pivotaltracker.com/story/show/60886000 https://www.pivotaltracker.com/story/show/60884508 https://www.pivotaltracker.com/story/show/60875916 https://www.pivotaltracker.com/story/show/60877478
Fixes API issues identified in these tickets:
https://www.pivotaltracker.com/story/show/60887336
https://www.pivotaltracker.com/story/show/60887246
https://www.pivotaltracker.com/story/show/60886000
https://www.pivotaltracker.com/story/show/60884508
https://www.pivotaltracker.com/story/show/60875916
https://www.pivotaltracker.com/story/show/60877478