Skip to content
This repository has been archived by the owner on Mar 13, 2024. It is now read-only.

Swap order between Romanian and Turkish #1559

Merged
merged 3 commits into from
Aug 14, 2018
Merged

Swap order between Romanian and Turkish #1559

merged 3 commits into from
Aug 14, 2018

Conversation

jasonblais
Copy link
Contributor

Romanian should be below Portuguese, above Turkish

Romanian should be below Portuguese, above Turkish
@jasonblais jasonblais added the 2: Dev Review Requires review by a core commiter label Aug 13, 2018
Copy link
Contributor

@enahum enahum left a comment

Choose a reason for hiding this comment

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

You only needed to change the order value, but This is fine

@cpanato
Copy link
Contributor

cpanato commented Aug 13, 2018

@jasonblais need to update the tests

@jasonblais
Copy link
Contributor Author

jasonblais commented Aug 13, 2018

If I just swap the order value and revert the other changes, do I need to update the tests?

@cpanato
Copy link
Contributor

cpanato commented Aug 13, 2018

@jasonblais no, but I can fix that if you want

@jasonblais
Copy link
Contributor Author

That would be helpful @cpanato :)

@cpanato
Copy link
Contributor

cpanato commented Aug 14, 2018

@enahum PTAL

Copy link
Contributor

@enahum enahum left a comment

Choose a reason for hiding this comment

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

LGTM

@jasonblais jasonblais added 4: Reviews Complete All reviewers have approved the pull request and removed 2: Dev Review Requires review by a core commiter labels Aug 14, 2018
@jasonblais
Copy link
Contributor Author

Thanks Carlos! Ready to merge (don't have permissions to merge)

@cpanato cpanato merged commit 769d307 into master Aug 14, 2018
@cpanato cpanato deleted the jasonblais-patch-1 branch August 14, 2018 17:27
@lindalumitchell lindalumitchell added the Tests/Not Needed Does not require new release tests label Aug 14, 2018
@amyblais amyblais added Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation labels Aug 17, 2018
fincha pushed a commit to fincha/mattermost-webapp that referenced this pull request Oct 21, 2018
* Swap order between Romanian and Turkish

Romanian should be below Portuguese, above Turkish

* Update i18n.jsx

* reorder and fix tests
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
4: Reviews Complete All reviewers have approved the pull request Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation Tests/Not Needed Does not require new release tests
Projects
None yet
5 participants