-
Notifications
You must be signed in to change notification settings - Fork 8
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
Remove DEI #2613
Remove DEI #2613
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, I no longer see the equity topic on the objective or filters. I also don't see the DEIA goal.
Abhorrent behavior. Do not comply in advance with fascism. |
404media yet again misconstruing the situation and failing to mention that this PR has a pending revert PR. Thanks for causing these developers a burden of a hate mob due to failing to keep journalistic integrity! |
@Cynosphere Ah, I was unaware of 404 media's coverage. I heard about this from a friend. Glad to see the pending revert PR. Still, why was it even removed in the first place? |
Scorched earth much? Watch the Federal data purge in real-time |
Description of change
Review the option for equity:
As part of this change, the topics filter select now queries the database for topics rather than referring to a separate string constant. This is simply a way of making sure that when we mark a topic as deletedAt, it is removed from all the relevant places.
How to test
Confirm equity has been removed from the places above.
I ask also that you scan the website for other places where we need to remove the forbidden words.
Issue(s)
Checklists
Every PR
Before merge to main
Production Deploy
After merge/deploy