-
Notifications
You must be signed in to change notification settings - Fork 108
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
Spanish #22
base: master
Are you sure you want to change the base?
Spanish #22
Conversation
confussion with "Sign Trasaction" = "Firmando transacción"; "Signed Transaction" = "Transacción firmada"; "Signing Transaction" = "Signing Transaction";
"/* suspicious error " 2 instances tagged
|
||
/* No comment provided by engineer. */ | ||
/* suspicious error repeated? signing transaction*/ | ||
"Sign Trasaction" = "Firmando transacción"; |
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.
Thank you for pointing out this misspelling. This is our fault. You may delete this line now. @zzaappedd
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.
suspicious error repeated? signing transaction
👍 Thank you for your PR. I'll try to compile and run your changes. As I can see by now, the project config file should also be edited. Don't worry I will edit this, and submit it with the merge. One more thing to mention, we usually work on the |
I've found some trivial issues.
I have already solved the There are only 2 steps left for you before we merging your PR.
When adding new commits to your repo on the |
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.
Hack
I know it has to have many mistakes. Got to learn to check for quotes and ; semicolon line endings with my text editor. Hopefully is not too bad.