Skip to content

Fix some translation issues#3656

Merged
UdjinM6 merged 2 commits into
dashpay:developfrom
UdjinM6:fix_some_tr
Aug 9, 2020
Merged

Fix some translation issues#3656
UdjinM6 merged 2 commits into
dashpay:developfrom
UdjinM6:fix_some_tr

Conversation

@UdjinM6
Copy link
Copy Markdown

@UdjinM6 UdjinM6 commented Aug 7, 2020

No description provided.

@UdjinM6 UdjinM6 added this to the 16 milestone Aug 7, 2020
Copy link
Copy Markdown
Member

@PastaPastaPasta PastaPastaPasta left a comment

Choose a reason for hiding this comment

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

utACK

@xdustinface
Copy link
Copy Markdown

Makes sense. Still one question: There are many places where the term "PrivateSend" is still included in a translated strings. Wouldn't it make sense then to just extract them all then if we are on it already?

@UdjinM6
Copy link
Copy Markdown
Author

UdjinM6 commented Aug 9, 2020

Extracting "PrivateSend" (or say "InstantSend") from more complex strings would make it hard for translators to understand/translate them imo (and they would have to translate such strings again while here we just slightly reduce the amount of work for them instead).

@UdjinM6 UdjinM6 merged commit d12137b into dashpay:develop Aug 9, 2020
xdustinface pushed a commit to xdustinface/dash that referenced this pull request Aug 26, 2020
* Avoid translating "PrivateSend"

* Make some strings translatable
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Sep 8, 2020
* Avoid translating "PrivateSend"

* Make some strings translatable
@UdjinM6 UdjinM6 deleted the fix_some_tr branch November 26, 2020 13:26
gades pushed a commit to cosanta/cosanta-core that referenced this pull request Mar 8, 2022
* Avoid translating "PrivateSend"

* Make some strings translatable
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.

3 participants