Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Extract text from CLI to enable internationalisation #182
Extract text from CLI to enable internationalisation #182
Changes from 7 commits
1413081
d828bb5
6e3a0a3
3924b16
5fceda2
b8c94a8
1f9626a
4f10175
b7f4188
d38323e
2b222cd
61e0c15
b288277
907bb2e
e439443
621ee4f
7b393bd
df09118
a84baf3
871b689
c61f197
a92890a
ad0b06b
a2f9a2a
7c99880
17a8840
63968ec
d607b06
7bbf7b3
6c2c708
25c093d
921b67d
c5ecde0
0542c1d
80f21e2
bcc64dd
7471ed6
6affab3
a7facf4
5bf8e54
9f09a2c
3a06e92
58a82d7
da97e6f
06d7920
e15806b
b31f814
68bb20d
ff34c4b
06470b0
902c279
b46e702
bd99461
126ebe2
64faf7e
261fdd9
0b8d1c4
7962d4c
d07aea6
6f388d8
12a6615
afad2b6
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Forwarding on a comment by @samajammin on initial intl audit (#180) that I agree with... consider making this clearer that the user is about to create a new password, vs typing an existing password, eg:
Create a password that will secure your validator keystore(s)
"Type the password that secures" kinda implies there is already password currently securing it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it possible for us to avoid including the tabs (
\t
) in the JSON files?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We might be able to ignore it in Crowdin but it'd be ideal to remove these to not confuse translators.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will look into extraction this from the json and having the tabs encoded in separate print statements. 👍