Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

Replace {} placeholders with ### and escape quotes in strings in POT file #152

Merged
merged 2 commits into from
Aug 24, 2021

Conversation

obulat
Copy link
Contributor

@obulat obulat commented Aug 24, 2021

Description

Some strings in the translation file have double quotes in them. These are illegal in PO files without escaping. This PR escapes the double quotes in the 'pot' conversion script.

@obulat obulat requested a review from a team as a code owner August 24, 2021 06:31
@obulat obulat requested review from zackkrida and sarayourfriend and removed request for a team August 24, 2021 06:31
@obulat obulat changed the title Escape quotes in strings in POT file Replace {} placeholders with ### and escape quotes in strings in POT file Aug 24, 2021
Copy link
Member

@zackkrida zackkrida left a comment

Choose a reason for hiding this comment

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

These improvements look good, thank you!

@zackkrida zackkrida merged commit 0a20921 into custom-pot-script Aug 24, 2021
@zackkrida zackkrida deleted the escape_quotes branch August 24, 2021 10:44
Copy link
Contributor

@sarayourfriend sarayourfriend left a comment

Choose a reason for hiding this comment

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

Looks good to me!

Comment on lines +4 to +13
/*
* white-space
# translator-comments
#. extracted-comments
#: reference…
#, flag…
#| msgid previous-untranslated-string
msgid untranslated-string
msgstr translated-string
*/
Copy link
Contributor

Choose a reason for hiding this comment

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

Thank you for adding this, very helpful!

@zackkrida zackkrida added the 🧰 goal: internal improvement Improvement that benefits maintainers, not users label Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🧰 goal: internal improvement Improvement that benefits maintainers, not users
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants