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

"Contact us" should have emailto link #631

Closed
wonderchook opened this issue Sep 1, 2016 · 7 comments
Closed

"Contact us" should have emailto link #631

wonderchook opened this issue Sep 1, 2016 · 7 comments
Labels
Milestone

Comments

@wonderchook
Copy link
Contributor

After resetting your password the messaging says "We have sent you an e-mail. Please contact us if you do not receive it within a few minutes." We should make "contact us" a link to email [email protected]

screenshot_20160901-154053

@nathalier
Copy link
Contributor

Hi!
I'd like to work on it as my first issue here:)

nathalier added a commit to nathalier/cadasta-platform that referenced this issue Sep 12, 2016
@nathalier
Copy link
Contributor

While I'm researching how to update translation (https://gitter.im/Cadasta/cadasta?at=57d6a3f383c1556511bd4c90), this is what I have so far: 5a95bd6 .
Any comments are appreciated!

@ian-ross
Copy link
Contributor

Ah, I'd forgotten that we'd already overridden the allauth templates. That should work fine, I think, and the strings all get pulled out for translation. This should be enough to fix this, so you could make a PR more or less immediately!

@nathalier
Copy link
Contributor

nathalier commented Sep 12, 2016

Do you mean it will be translated soon? Without fix, localized version of the phrase is shown in other languages. But the fix breaks it (because of changed msgid), and phrase in English is shown instead.

@wonderchook
Copy link
Contributor Author

@nathalier translations are done in Transifex, so first the fix would happen in English and then be translated later.

@ian-ross
Copy link
Contributor

Yeah, that's OK. You just need to change the source language text in the templates (which you've done). Then we pull all the messages out, upload them to Transifex, someone translates them, then we download the translations and Django picks up the appropriate translation from the message catalogues. The process is described here: https://devwiki.corp.cadasta.org/Translation

@nathalier
Copy link
Contributor

@wonderchook @ian-ross Thanks for guidance! :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants