-
Notifications
You must be signed in to change notification settings - Fork 51
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
Rename mail and change order #44
base: development
Are you sure you want to change the base?
Rename mail and change order #44
Conversation
Co-authored-by: Marco A. Gutierrez <[email protected]>
Co-authored-by: Marco A. Gutierrez <[email protected]>
Co-authored-by: Marco A. Gutierrez <[email protected]>
…ventyay-talk into mail_rename_order
Do I have to change anything in this PR? @marcoag |
Yes, I think some suggestion I made on #35 might have not been applied here, so please have a look, i.e.: e-mail or email consistency (choose one and use it through the changes)
|
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.
Requesting changes, please take a look at my comment above.
I'm pretty sure I had implemented all of your suggestion in the previous PR, but I'll still have a look. But for changing email/mail translation and the message center, how do I decide what's right? should I just google translate and make the change? |
You're mixing thins:
Since you missed this suggestion I asked you to also double check no others are missing here.
Which basically changes the translation |
For the Portuguese from portugal weird translation you can probably use some decent online translation tool. |
online tools are suggesting the same translations for Portuguese (Portugal), except some which suggest 'Redigir Emails' instead of 'Compor Emails' for Compose. And as for the discrepancy between email and e-mail, even other translations are very random between the two |
Ok for portuguese. For the e-email vs email check the most common one around and use that one in this PR, at least lets start to be consistent as long as we can. And resolve conflict, plz. Also please check for other appearances of the modified |
I checked and saw email is more common, so used email in my changes. |
@marcoag anything else to be changed? |
This PR closes/references issue #11
How has this been tested?
Results on running the server locally are:
Checklist
doc/changelog.rst
.