Skip to content

Chore: Fix i18n duplicated keys#19998

Merged
sampaiodiego merged 2 commits into
developfrom
fix-i18n-duplicated-keys
Dec 30, 2020
Merged

Chore: Fix i18n duplicated keys#19998
sampaiodiego merged 2 commits into
developfrom
fix-i18n-duplicated-keys

Conversation

@sampaiodiego

Copy link
Copy Markdown
Member

Proposed changes (including videos or screenshots)

LingoHub doesn't like duplicated keys on JSON files, which cause an import to fail, making things not being synced, so I'm adding a validation of duplicated keys to prevent that from happen again.

Issue(s)

Steps to test or reproduce

Further comments

@sampaiodiego sampaiodiego merged commit 11e2a9e into develop Dec 30, 2020
@sampaiodiego sampaiodiego deleted the fix-i18n-duplicated-keys branch December 30, 2020 14:26
gabriellsh added a commit that referenced this pull request Jan 4, 2021
…info_page

* 'develop' of github.com:RocketChat/Rocket.Chat: (255 commits)
  Chore: Add tests for the api/licenses.* endpoints (#20041)
  Language update from LingoHub 🤖 on 2021-01-04Z (#20034)
  [FIX] Omnichannel Agents unable to take new chats in the queue (#20022)
  [FIX] Omnichannel Business Hours form is not being rendered (#20007)
  Language update from LingoHub 🤖 (#20013)
  Chore: Fix i18n duplicated keys (#19998)
  [FIX] Agent information panel not rendering (#19965)
  [FIX] Apps Create Room method with agent not working (#19997)
  Chore: add tests to api/instances.get endpoint  (#19988)
  Bump version to 3.11.0-develop
  Bump version to 3.10.0
  Bump version to 3.10.0-rc.4
  Bump version to 3.10.0-rc.3
  Bump version to 3.10.0-rc.2
  Bump version to 3.10.0-rc.1
  Bump version to 3.10.0-rc.0
  Revert "[NEW] Storj integration (#19932)"
  Regression: polishing licenses endpoints  (#19981)
  Regression: Double Scrollbars on tables (#19980)
  Regression: Add currently running instance to instances.get endpoint (#19955)
  ...
@sampaiodiego sampaiodiego mentioned this pull request Feb 1, 2021
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.

2 participants