-
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
Update how link header works in CollectionService #149
Comments
Hi, i have been trying to get a good notion of existing "rel" values, and i shoud ask @Islandora-CLAW/7-x-2-x-sprinters about this: "member" is there but with a different meaning, Suggestions? |
Since the link involves an islandora service resource endpoint, we could in the future return instead the members service endpoint( putting new members for example) using something like this |
Well, it surely isn't |
@ruebot, so we go for |
Just based on how Apple and DCTerms are specified in that list, should we consider something like |
I would be very careful about adding new |
@acoburn, you are right, that is the reason i added the list in the first place. In the iana world, there is relation named |
@DiegoPino I like your comment about providing the other relevant API endpoints instead of LDP specifics, which are really just behind the scenes implementation details. We just need to figure out those endpoints! But I definitely could see us providing links to a endpoint that would list all members for a collections. And definitely the add/remove member endpoints too. |
@daniel-dgi++, we will know more about this when we finish those endpoints. Good to know we are in sync with this.
could be also a free interpretation of an indirect/direct container. LDP should have their own 😄 |
@ruebot, member is not in iana's list, but it's valid one. If we go for iana i would suggest for now "hub" |
I'm good with hub. @Islandora-CLAW/7-x-2-x-committers what do y'all think? |
@whikloj, @daniel-dgi are we good with hub? If so, then @DiegoPino can update #152 and we should be good for a merge there. |
hub's fine for now. i'm sure we'll go back and forth on this a lot over time. but let's merge it in! i like it. |
Ok, updated #152, hub it is. |
Resolved with 238cfb4 |
Should provide a link header that links to the membership in the indirect container with a rel of members.
The text was updated successfully, but these errors were encountered: