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

Placeholders are not replaced. #3650

Merged
merged 2 commits into from
Jun 30, 2016

Conversation

peccu
Copy link
Contributor

@peccu peccu commented Jun 29, 2016

@RocketChat/core

Placeholders in mail template are also translated. They are not replaced.

@sampaiodiego
Copy link
Member

LGTM

@peccu can you please rebase and fix the conflicts? thx

@peccu
Copy link
Contributor Author

peccu commented Jun 30, 2016

Thank you for review.
I fixed the conflicts.

and, can I join Lingohub?

@sampaiodiego
Copy link
Member

Sure @peccu .. Please send an email to [email protected] containing the language you want to help translanting and an email address to invite.

Thank you again.

@sampaiodiego sampaiodiego merged commit 6c6d83e into RocketChat:develop Jun 30, 2016
@peccu peccu deleted the feature/correct-placeholders branch July 28, 2016 02:55
@marceloschmidt marceloschmidt added this to the 0.36.0 milestone Sep 5, 2016
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.

3 participants