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.

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

[CC License support] Addingresponse to "rightsByQuestions" #129

Merged
merged 4 commits into from Jun 17, 2020

Conversation

KevinVdV
Copy link
Member

@KevinVdV KevinVdV commented Jun 9, 2020

As mention in the PR: DSpace/DSpace#2764, this PR will clarify the response of the /api/config/submissioncclicenses/search/rightsByQuestions endpoint.

@KevinVdV KevinVdV mentioned this pull request Jun 9, 2020
6 tasks
Copy link
Member

@tdonohue tdonohue left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@KevinVdV : I've added feedback inline on these updated docs. Thanks for starting these. Overall, they look good, but I think we need to provide more hints to the client on how to use this endpoint. See inline notes below

submissioncclicenses.md Outdated Show resolved Hide resolved
submissioncclicenses.md Outdated Show resolved Hide resolved
submissioncclicenses.md Outdated Show resolved Hide resolved
submissioncclicenses.md Outdated Show resolved Hide resolved
Copy link
Member

@tdonohue tdonohue left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Looks good to me for now. As noted in a separate comment, we (meaning the DSpace 7 team) may wish to analyze where we are creating new Submission-feature endpoints. We have an inconsistency in that some are under /api/config/ (prefixed by "submission"), while others are under /api/submission/. I suspect we may want to move most/all to the latter path at some point, but that could be done in a follow-up PR.

@tdonohue
Copy link
Member

Merging this as the change here is really tiny & based on my feedback in the implementation PR at DSpace/DSpace#2764

@tdonohue tdonohue merged commit 983508e into DSpace:master Jun 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants