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

Send norent letter in English, store in user's locale. #1473

Merged
merged 4 commits into from
May 22, 2020

Conversation

toolness
Copy link
Collaborator

@toolness toolness commented May 21, 2020

This fixes #1246 by forcing the current locale to English when creating/sending the NoRent letter and email to the landlord. We also store the letter in the user's current locale, so that they can always read it later in the language they understand.

To do

  • Add tests.

@toolness toolness mentioned this pull request May 21, 2020
33 tasks
@toolness toolness marked this pull request as ready for review May 21, 2020 11:02
@toolness toolness requested a review from sraby May 21, 2020 14:49
@toolness
Copy link
Collaborator Author

Hey @sraby I've asked for your review on this but I know it's almost entirely Python code you're unfamiliar with--still, I'd like to walk you through the basics of what it's doing and make sure it makes sense to you!

@toolness toolness merged commit d752156 into master May 22, 2020
@toolness toolness deleted the send-norent-letter-in-english branch May 22, 2020 09:56
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.

norentSendLetter mutation should force locale to English
1 participant