-
Notifications
You must be signed in to change notification settings - Fork 318
Add license #81
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
Add license #81
Conversation
I've added custom licenses now, as well as a link to the Open Definition site |
Wow, clever approach. Thanks for fitting it within the pattern established by the undocumented javascript architecture! Took me a few minutes to figure out how the hidden fields were working. It dawned on me that this could also be a lot simpler if we didn't need to store the |
FWIW, use decided to use a license URL for the data.json standard because that was the best model that we could devise that was a) simple b) extensible and c) non-prescriptive. (At least, as I recall. It's been a few years.) You may well decide that you'd like to be prescriptive, or that you prefer something more complicated—we were trying to establish a government standard, so our requirements were rather different than yours need be. :) |
Hey @pezholio just wanted to check in on this. I'd love to include this in |
Yep, I've got some time scheduled in this coming week to do some work on this, so will get cracking on this tomorrow. I like the idea of only storing the URL, but you're right that this might cause issues when people choose the 'other' option. That said, we might want to do away with the other option and assume that if people want to add extra licenses, they can do this via the yaml. What do you think? Also, a further improvement down the line might be to let people manage this yaml file (and other similar files) via a GUI. |
I've rebased this branch against the |
I've now added a section to the I had to hack around with the liquid markup of the text form include, as it seemed like it wanted an object to populate the form label, so I've added a default fallback. There's also a hidden field in the licenses list that makes sure Any feedback, please let me know! |
This looks great @pezholio! Really top job. Thanks for rebasing and again fitting it into the new architecture -- hope you found this one a little easier. I've made a couple tiny alterations:
Merging now. Thanks again for your contribution! |
Awesome, glad you like. I'll be keeping an eye out for any contributions I can make in the future 👍 |
This adds a license field to the edit page, allowing the user to choose a license from a (currently) limited set. Next step is to allow users to select the
other
option and specify their own license if they want.Might also be worth linking to http://licenses.opendefinition.org/ to give some options.