-
Notifications
You must be signed in to change notification settings - Fork 54
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 license in .JSON files from Disconnect #223
Comments
Hey @patjack! While reviewing Shavar services I realized we did not update the license in the disconnect-blocklist.json and there was never a license key-value pair on the disconnect-entitylist.json. Can you please guide us/add a PR to update the licenses of these files? P.S. I checked the Disconnect repo for the entities.json to see if you all had license on that but you all also did not have license key-value pair for entities. cc @englehardt |
Hi @say-yawn, Thanks for the ping on this! Yes, we should add the license to this file. I'm thinking we can follow the same form as the blacklist and have entities moved to an
If that sounds good to you I can submit a PR w/these changes. Thanks! |
Yes that sounds great! Thanks for following-up. It seems I'll have to make some changes to how we handle entities on the list creation script, would it be possible to create two separate PRs for adding the licenses in blocklist and entitylist respectively? |
No problem! Those two PRs will be arriving shortly. |
As mentioned in mozilla-services/shavar-prod-lists#223 we now have a new structure on entities list
As mentioned in mozilla-services/shavar-prod-lists#223 we now have a new structure on entities list
As mentioned in mozilla-services/shavar-prod-lists#223 we now have a new structure on entities list
All the changes to licenses are now reflected on main and versioned branches. |
About this Issue
After #211 we have decided to move to
Attribution-NonCommercial-ShareAlike 4.0 International
. We should update the license key value pair in .JSON files from DisconnectAcceptance Criteria
The text was updated successfully, but these errors were encountered: