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

AO3-5351 Make remaining text in feedback response i18n-able #4788

Merged
merged 5 commits into from
May 19, 2024

Conversation

brianjaustin
Copy link
Member

Pull Request Checklist

Issue

https://otwarchive.atlassian.net/browse/AO3-5351

Purpose

I18n-ising the remaining text in the feedback (support ticket) response emails. I didn't add a preview since I did that already in #4772.

Credit

Brian Austin (they/he)

@Bilka2
Copy link
Contributor

Bilka2 commented Apr 10, 2024

It would be nice if you added it_behaves_like "a translated email" to the user mailer spec for the email.

@brianjaustin
Copy link
Member Author

@Bilka2 indeed, thanks! I've added those

@brianjaustin brianjaustin added Reviewed: Ready to Merge Scope: i18n Only Only moves text to en.yml for i18n purposes and removed Coder Has Actioned Review labels Apr 11, 2024
Copy link
Member

@sarken sarken left a comment

Choose a reason for hiding this comment

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

While we're here, can we rename feedback_response to just plain feedback in the mailer preview? Translation said it's a bit less confusing if the preview name matches the key for the mailer when possible.

@sarken sarken merged commit 9d9c832 into otwcode:master May 19, 2024
26 checks passed
@brianjaustin brianjaustin deleted the AO3-5351-i18n-feedback branch May 21, 2024 21:33
sarken added a commit that referenced this pull request May 31, 2024
sarken added a commit that referenced this pull request May 31, 2024
…4824)

Revert "AO3-5351 Make remaining text in feedback response i18n-able (#4788)"

This reverts commit 9d9c832.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Reviewed: Ready to Merge Scope: i18n Only Only moves text to en.yml for i18n purposes
Projects
None yet
3 participants