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

User left message token replacement not working #467

Merged
merged 1 commit into from
Aug 15, 2015

Conversation

rwakida
Copy link
Contributor

@rwakida rwakida commented Aug 15, 2015

Fix for #466

Replaces gender specific translation with non-gender translation.

Grammatical gender change from 6329d07
references 'gender' message field. If field doesn't exist, then
username is not replaced.

Gender specific messages don't seem to be implemented. The
6329d07 commit hardcoded "male" value
in other places that used gender specific translation.

Replaced gender specific translation with non-gender specific translation, and
removed gender option.

Grammatical gender change from 6329d07
references 'gender' message field.  If field doesn't exist, then
username is not replaced.

Replaced gender specific translation with non-gender specific call, and
removed gender option.

Gender specific messages don't seem to be implemented.  The
6329d07 commit hardcoded "male" value
in other places that used gender specific translation.
@rwakida rwakida changed the title Replaces gender specific translation with non-gender translation. User left message token replacement not working Aug 15, 2015
rodrigok added a commit that referenced this pull request Aug 15, 2015
User left message token replacement not working
@rodrigok rodrigok merged commit 8e0f5e3 into RocketChat:master Aug 15, 2015
@rodrigok
Copy link
Member

Thanks @rwakida

Peym4n pushed a commit to redlink-gmbh/Rocket.Chat that referenced this pull request Sep 10, 2018
* add threads as new room type t to statistic

* add some comments + changed roomtype from t to thread
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.

None yet

2 participants