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

Change gendered language to gender-neutral 'they/them/their' #25505

Merged
merged 1 commit into from
Nov 3, 2020

Conversation

sarina
Copy link
Contributor

@sarina sarina commented Nov 2, 2020

There was one user-facing string with gendered language, as well as a number of comments/docstrings throughout the codebase.

per https://openedx.atlassian.net/browse/TNL-7560

@edx-status-bot
Copy link

Your PR has finished running tests. There were no failures.

Copy link
Contributor

@matthugs matthugs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We may want to check to see whether we can search the alert conditions that exist in New Relic for searches that match the error message texts. But if we would have to check things linearly, probably not worth the time.

@sarina
Copy link
Contributor Author

sarina commented Nov 3, 2020

@matthugs good idea. I did a quick search through NR and didn't find any alerts or anything that referenced this exact wording, but checking policies is a linear check.

@sarina sarina merged commit 18059c2 into master Nov 3, 2020
@sarina sarina deleted the sarina/gender-neutral-language branch November 3, 2020 21:17
@edx-pipeline-bot
Copy link
Contributor

EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production.

@edx-pipeline-bot
Copy link
Contributor

EdX Release Notice: This PR has been deployed to the production environment.

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.

4 participants