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

Update license in .JSON files from Disconnect #223

Closed
4 tasks done
say-yawn opened this issue Jun 29, 2020 · 5 comments
Closed
4 tasks done

Update license in .JSON files from Disconnect #223

say-yawn opened this issue Jun 29, 2020 · 5 comments

Comments

@say-yawn
Copy link
Contributor

say-yawn commented Jun 29, 2020

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 Disconnect

Acceptance Criteria

@say-yawn
Copy link
Contributor Author

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

@patjack
Copy link
Contributor

patjack commented Jun 29, 2020

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 entities key. Like this:

{

"license": "Copyright 2010-2020..."
"entities" : { }
}

If that sounds good to you I can submit a PR w/these changes. Thanks!

@say-yawn
Copy link
Contributor Author

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?

@patjack
Copy link
Contributor

patjack commented Jun 30, 2020

No problem! Those two PRs will be arriving shortly.

say-yawn added a commit to mozilla-services/shavar-list-creation that referenced this issue Jul 14, 2020
As mentioned in mozilla-services/shavar-prod-lists#223 we now have a new structure on entities list
say-yawn added a commit to mozilla-services/shavar-list-creation that referenced this issue Jul 14, 2020
As mentioned in mozilla-services/shavar-prod-lists#223 we now have a new structure on entities list
say-yawn added a commit to mozilla-services/shavar-list-creation that referenced this issue Jul 14, 2020
As mentioned in mozilla-services/shavar-prod-lists#223 we now have a new structure on entities list
@say-yawn
Copy link
Contributor Author

All the changes to licenses are now reflected on main and versioned branches.

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

No branches or pull requests

2 participants