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

i18n: Remove a duplicate message key. #4902

Merged
merged 1 commit into from
Jul 17, 2021

Conversation

chrisbobbe
Copy link
Contributor

I got the following error when I ran tools/tx-sync:

Step 3: Upload strings to translate...
+ tx --quiet push -s
tx ERROR: Error received from server: Duplicate string key ('Oops! Something went wrong\.') in line 247
tx ERROR: Could not upload source file. You can use --skip to ignore this error and continue the execution.
tx ERROR: Error received from server: Duplicate string key ('Oops! Something went wrong\.') in line 247

Looks like maybe a rebase error when I was working on #4754. The
string was already added, in #4829.

@chrisbobbe chrisbobbe added the P0 critical Highest priority label Jul 16, 2021
@chrisbobbe
Copy link
Contributor Author

P0 because this is an easy fix for a problem that's blocking a release. I guess I also don't know what consequences this duplication might have at runtime; I haven't yet tested. 🙂

I got the following error when I ran `tools/tx-sync`:

```
Step 3: Upload strings to translate...
+ tx --quiet push -s
tx ERROR: Error received from server: Duplicate string key ('Oops! Something went wrong\.') in line 247
tx ERROR: Could not upload source file. You can use --skip to ignore this error and continue the execution.
tx ERROR: Error received from server: Duplicate string key ('Oops! Something went wrong\.') in line 247
```

Looks like maybe a rebase error when I was working on zulip#4754. The
string was already added, in zulip#4829.
@gnprice gnprice merged commit 71b7c2c into zulip:master Jul 17, 2021
@gnprice
Copy link
Member

gnprice commented Jul 17, 2021

Looks good, thanks -- merging.

@chrisbobbe chrisbobbe deleted the pr-duplicate-i18n-message branch November 4, 2021 21:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P0 critical Highest priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants