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] Added zip-ties tag to incidents (using text search only) #833

Merged
merged 3 commits into from
Jul 7, 2020
Merged

[UPDATE] Added zip-ties tag to incidents (using text search only) #833

merged 3 commits into from
Jul 7, 2020

Conversation

hossain-khan
Copy link
Collaborator

The zip-ties tag is added to incidents which mentioned this either in title or description.

The `zip-ties` tag is added to incidents which mentioned this either in title or description.

- Tag is added as a part of effort mentioned in #810
- The tag has been officially added in https://github.com/2020PB/police-brutality/pull/832/files
@emarcey
Copy link
Collaborator

emarcey commented Jul 6, 2020

This is a good first pass, but I'd go through again. I know at least ca-losangeles-32 and ca-santamonica-2 also have zip ties

@hossain-khan
Copy link
Collaborator Author

@emarcey Thanks for pointing out, did another pass, and updated PR.

@emarcey
Copy link
Collaborator

emarcey commented Jul 7, 2020

Are you doing a ctrl-F for "zip ties" or "zip-ties" or something. My one concern here is that police use zip ties for the vast, vast majority of protest-related arrests, because they can carry loads of them at a time. So I'm worried that by just searching on zip tie things, we're missing tons of incidents where zip ties are used

@hossain-khan
Copy link
Collaborator Author

hossain-khan commented Jul 7, 2020

Yes, that is correct, I have done a project-wide search for key terms like zip, zip-ties, zip tied, tied and so on.

Wanted to cover the most obvious case first. You can keep the issue #810 open if anybody wants to go though each incident links and update the tags. Or, if you want this to happen in one go via that process, I am fine with that too. In that case you can close this PR. Thanks 🙏

@hossain-khan hossain-khan changed the title [UPDATE] Added zip-ties tag to incidents [UPDATE] Added zip-ties tag to incidents (using text search only) Jul 7, 2020
@emarcey
Copy link
Collaborator

emarcey commented Jul 7, 2020

No I think it's fine for a first pass, thanks!

@emarcey emarcey merged commit 7070cb4 into 2020PB:master Jul 7, 2020
@hossain-khan hossain-khan deleted the feature/810-add-zip-ties-tag branch July 7, 2020 20:13
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

Successfully merging this pull request may close these issues.

2 participants