-
Notifications
You must be signed in to change notification settings - Fork 56
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
Rights/license registry #2248
Merged
Merged
Rights/license registry #2248
Changes from all commits
Commits
Show all changes
11 commits
Select commit
Hold shift + click to select a range
62986ca
add existing cc and rights statements to registry
kirschbombe b6f21b0
add rights to registry home
kirschbombe b09de0c
fix line break in links
kirschbombe 2e087b8
update rights registry title
kirschbombe 6296adb
fix yaml indents
kirschbombe 00a6551
add cc and rights URIs and note
kirschbombe 9734512
fix presi3 rights url
kirschbombe 51b51a5
Merge branch 'main' into license-registry
kirschbombe d923172
add Julie to editors
kirschbombe f4cdd7f
add Julie orcid
kirschbombe f8cb7e5
update editor affiliations
kirschbombe File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
This one is important. Either we need to list all of the CC / RSs licenses or not list them and keep this assertion. The URIs in the PR aren't themselves licenses/statements.
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.
I like the idea of listing these two standards (esp. since we have no other to list at the moment), but could we have a column for each that says "for a full list of the CC license URIs, see...." rather than listing each license/statement URI? Then we don't have to worry about updating if the specific licenses/statements change over time -- and don't have to list them all here when they are listed elsewhere. Or we could list a few of the most common, then send folks to CC and RS for more.