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 ua.csv #289

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Update ua.csv #289

wants to merge 4 commits into from

Conversation

kermoshina
Copy link
Contributor

Added new urls based on this document from SBU (Security Service Ukraine) that lists recommendations for ISPs to block these websites "against Russian propaganda". http://inau.ua/sites/default/files/file/1801/vhidnyy_no_08_vid_11.01.2018.pdf
These websites mainly belong to TV channels, including scientific channels and channels for kids. This list is likely to be blocked within governmental institutions and/or universities and schools.

xeniax added 2 commits January 27, 2018 15:03
Added new urls based on this document from SBU (Security Service Ukraine) that lists recommendations for ISPs to block these websites  "against Russian propaganda". http://inau.ua/sites/default/files/file/1801/vhidnyy_no_08_vid_11.01.2018.pdf 
These websites mainly belong to TV channels, including scientific channels and channels for kids. This list is likely to be blocked within governmental institutions and/or universities and schools.
@hellais
Copy link
Collaborator

hellais commented Apr 19, 2018

This is now out of sync with the master branch. You should re-open a new PR with the additional URLs in it and close this one.

@agrabeli
Copy link
Collaborator

@hellais @sneft I edited the category codes and fixed the PR so that there are no longer conflicts with the base branch. I think the missing thing to do is to add slashes to a bunch of URLs (see Travis failed checks). Do you perhaps have a script for that?

@agrabeli
Copy link
Collaborator

@hellais can you please help with fixing this PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants