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

Retrieve feedback Strings from Resources #1194

Merged
merged 1 commit into from
Aug 6, 2018
Merged

Conversation

danesfeder
Copy link
Contributor

Closes #1193

@danesfeder danesfeder added bug Defect to be fixed. ✓ ready for review labels Aug 6, 2018
@danesfeder danesfeder added this to the 0.17.0 milestone Aug 6, 2018
@danesfeder danesfeder self-assigned this Aug 6, 2018
@danesfeder
Copy link
Contributor Author

device-2018-08-06-105228

@danesfeder danesfeder force-pushed the dan-feedback-strings branch 4 times, most recently from cebd03d to 0962761 Compare August 6, 2018 15:18
@danesfeder danesfeder force-pushed the dan-feedback-strings branch from 0962761 to a9345c0 Compare August 6, 2018 15:22
@Guardiola31337
Copy link
Contributor

Guardiola31337 commented Aug 6, 2018

Noting here that we should 👀 in Transifex the translated strings that include control characters (e.g. new line \n) before accepting them. That's not the case for feedback_not_allowed German translation (Nicht erlaubt).

Copy link
Contributor

@Guardiola31337 Guardiola31337 left a comment

Choose a reason for hiding this comment

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

Noting here that we should 👀 in Transifex the translated strings that include control characters (e.g. new line \n) before accepting them. That's not the case for feedback_not_allowed German translation (Nicht erlaubt).

We should check next time that we pull down the latest translations from Transifex.

@danesfeder danesfeder merged commit 9299814 into master Aug 6, 2018
@danesfeder danesfeder deleted the dan-feedback-strings branch August 6, 2018 16:08
@Guardiola31337 Guardiola31337 mentioned this pull request Aug 10, 2018
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Defect to be fixed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants