-
Notifications
You must be signed in to change notification settings - Fork 10
Get Collections For Linking Use Case #367
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
Conversation
…r number for dataset and collection identifiers
…candidates for unlinking
ekraffmiller
left a comment
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.
looks good, I just found some places where I think 'dataverse' should be replaced with 'collection'?
|
thanks @ekraffmiller , changes done |
ekraffmiller
left a comment
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.
looks good!
|
tests passing - merging |
What this PR does / why we need it:
Adds a new Use Case - Get Collections For Linking.
It can receive an optional parameter to return collections for unlinking instead.
Which issue(s) this PR closes:
Related Dataverse PRs:
Suggestions on how to test this:
Check test passing.