-
Notifications
You must be signed in to change notification settings - Fork 71
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
Sprint 003 Thin Collection Controller #160
Sprint 003 Thin Collection Controller #160
Conversation
Have the idToUri return a 404 instead of halting execution
Included a script to align the git branches required by the various composer.json files
@ruebot, @DiegoPino, @daniel-dgi, @nigelgbanks, @edf This adds a working vagrant with a new port :8282 which has access to the endpoints
Please try it out and see if the vagrant comes up for you and you are able to test these end points. |
building |
That could be concerning, but it looks like everything built fine. But....
|
Probably should drop the binary and archive files? phar, etc? |
@ruebot ok, I will try a fresh clone |
Building fresh clone... |
Built and: vagrant@islandora-deux:~$ curl -i localhost:8282/islandora/resource vagrant@islandora-deux:~$ curl -i localhost:8282/islandora/transaction |
@whikloj collection seems funky. ...but that was a build before 3b98597... so I'll try and build again before I leave for the airport.
|
...and facepalm. There isn't a GET method for the collection service. @whikloj I'm good with merging this as is if @DiegoPino is cool with it. Then we can continue building and create another PR. |
This PR subsumes #159 and daniel-dgi/islandora#2
If those are merged this is not needed.
Includes @daniel-dgi changes to a thin Collection controller.
Includes addition and deletion of proxy objects for pcdm:objects.
Adds a response parse to get the transaction ID.
Changes ResourceService's idToUri function to return a 404 response instead of halting all processing.