-
-
Notifications
You must be signed in to change notification settings - Fork 852
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
[strings] Update editor_edits_sent_message string #6993
[strings] Update editor_edits_sent_message string #6993
Conversation
dc91734
to
db568b6
Compare
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.
Thanks! Calling for reviews from @organicmaps/translations
c228069
to
b6b8b63
Compare
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.
ok for nl
b6b8b63
to
8c5bac8
Compare
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.
Hu needs changes
@@ -811,7 +811,7 @@ | |||
|
|||
"placepage_delete_bookmark_button" = "Könyvjelző törlése"; | |||
|
|||
"editor_edits_sent_message" = "Javasolt változtatásainak elküldése sikerült"; | |||
"editor_edits_sent_message" = "Az Ön megjegyzése elküldésre kerül az OpenStreetMap-nek"; |
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.
"editor_edits_sent_message" = "Az Ön megjegyzése elküldésre kerül az OpenStreetMap-nek"; | |
"editor_edits_sent_message" = "Az Ön változtatási javaslata elküldésre fog kerülni"; |
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.
Hm, DeepL says that Az Ön változtatási javaslata elküldésre került
means Your change proposal has been sent
.
But we want to emphasize that change is not sent yet, it will be sent at some time in the future.
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.
@d4f5409d happy new year! Waiting for your response here.
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.
HNY! Don't rely on DeepL that much, it isn't 100% perfect ;)
I changed the translation accordingly to your review.
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.
updated ✔️
8c5bac8
to
edeb5b7
Compare
@biodranik hi! Are we waiting for more reviews or can proceed with merging this? |
@biodranik PTAL |
@MontakOleg can you rebase your branch? |
Signed-off-by: Oleg Montak <oleg.montak@gmail.com>
edeb5b7
to
08171f7
Compare
@Jean-BaptisteC yep, rebased. |
08171f7
to
904c456
Compare
@biodranik ready to merge |
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!
@MontakOleg can you please squash/fixup two last commits before the merge? |
Signed-off-by: Oleg Montak <oleg.montak@gmail.com>
904c456
to
c9c5ca6
Compare
@biodranik Sure! Done. |
Update
editor_edits_sent_message
string to reflect real app behavior (updates will be sent when the app goes to the background), as suggested in comment #6986 (comment)