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

[FIX] Translation interpolations for many languages #13751

Merged
merged 26 commits into from
Mar 19, 2019

Conversation

fliptrail
Copy link
Contributor

Closes #12130, Closes #12001

Fixes these and some string modifier issues.

Initial State

imported

Final State

Screenshot from 2019-03-17 00-22-40

@engelgabriel engelgabriel added this to the 1.0.0 milestone Mar 18, 2019
@engelgabriel
Copy link
Member

engelgabriel commented Mar 18, 2019

@sampaiodiego do we need to upload the files directly to LingoHub once this is merged? Or are we still doing the auto-fetch from here?

@sampaiodiego
Copy link
Member

@engelgabriel it is currently turned off for languages other than english.. I'll turned it on again as we're already missing some translations there.

Copy link
Member

@sampaiodiego sampaiodiego left a comment

Choose a reason for hiding this comment

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

there is no need to do any changes on cleanHistory.js file.. the usage of the t function is correct and working there.

all other changes on translation files are welcome.

@fliptrail
Copy link
Contributor Author

@sampaiodiego Please review now.

Copy link
Member

@sampaiodiego sampaiodiego left a comment

Choose a reason for hiding this comment

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

there are many others where Prune_Warning_all became %srning_all

packages/rocketchat-i18n/i18n/ru.i18n.json Outdated Show resolved Hide resolved
packages/rocketchat-i18n/i18n/ru.i18n.json Outdated Show resolved Hide resolved
packages/rocketchat-i18n/i18n/ru.i18n.json Outdated Show resolved Hide resolved
packages/rocketchat-i18n/i18n/ru.i18n.json Outdated Show resolved Hide resolved
packages/rocketchat-i18n/i18n/sk-SK.i18n.json Outdated Show resolved Hide resolved
packages/rocketchat-i18n/i18n/sk-SK.i18n.json Outdated Show resolved Hide resolved
packages/rocketchat-i18n/i18n/cy.i18n.json Outdated Show resolved Hide resolved
packages/rocketchat-i18n/i18n/cy.i18n.json Outdated Show resolved Hide resolved
packages/rocketchat-i18n/i18n/cy.i18n.json Outdated Show resolved Hide resolved
packages/rocketchat-i18n/i18n/cy.i18n.json Outdated Show resolved Hide resolved
sampaiodiego and others added 7 commits March 19, 2019 03:21
Co-Authored-By: fliptrail <40834245+fliptrail@users.noreply.github.com>
Co-Authored-By: fliptrail <40834245+fliptrail@users.noreply.github.com>
Co-Authored-By: fliptrail <40834245+fliptrail@users.noreply.github.com>
Co-Authored-By: fliptrail <40834245+fliptrail@users.noreply.github.com>
Co-Authored-By: fliptrail <40834245+fliptrail@users.noreply.github.com>
Co-Authored-By: fliptrail <40834245+fliptrail@users.noreply.github.com>
Co-Authored-By: fliptrail <40834245+fliptrail@users.noreply.github.com>
sampaiodiego and others added 5 commits March 19, 2019 03:22
Co-Authored-By: fliptrail <40834245+fliptrail@users.noreply.github.com>
Co-Authored-By: fliptrail <40834245+fliptrail@users.noreply.github.com>
Co-Authored-By: fliptrail <40834245+fliptrail@users.noreply.github.com>
Co-Authored-By: fliptrail <40834245+fliptrail@users.noreply.github.com>
Co-Authored-By: fliptrail <40834245+fliptrail@users.noreply.github.com>
@fliptrail
Copy link
Contributor Author

Looks like there was a problem with my RegEx. Will see to it tomorrow.

@sampaiodiego sampaiodiego changed the title [FIX] Debugs clearHistory.js to make it work for all languages [FIX] Translation interpolations for many languages Mar 19, 2019
@sampaiodiego sampaiodiego merged commit c143bd4 into RocketChat:develop Mar 19, 2019
@rodrigok rodrigok mentioned this pull request Apr 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants