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

Add support for compiling strings from JavaScript expressions in filters #29

Merged
merged 1 commit into from
Dec 18, 2017

Conversation

mtr
Copy link
Contributor

@mtr mtr commented Dec 18, 2017

For example, presented with the fragment
<span ng-bind="isNight ? 'Moon' + 'shine' : 'Day' + 'light' |translate"></span>
the extractor will produce both Moonshine and Daylight.

Additionally, efforts have been made to make sure escaped quotes in the source
are handled correctly, so that the whole round-trip from extracting translatable strings
from the source through translating them into .po files and looking them up during
runtime works as expected, with strings like 'Five o\'clock'.

…ers.

For example, presented with the fragment
`<span ng-bind="isNight ? 'Moon' + 'shine' : 'Day' + 'light' |translate"></span>`
the extractor will produce both `Moonshine` and `Daylight`.

Additionally, efforts have been made to make sure escaped quotes in the source
are handled correctly, so that the whole round-trip from extracting translatable strings
from the source through translating them into .po files and looking them up during
runtime works as expected, with strings like `'Five o\'clock'`.
@mtr
Copy link
Contributor Author

mtr commented Dec 18, 2017

I'll do as I've been encouraged to, and merge this pull request myself. 😄

Can you please provide a new NPM build @vperron?

@mtr mtr merged commit cc4957d into Polyconseil:master Dec 18, 2017
@mtr
Copy link
Contributor Author

mtr commented Dec 20, 2017

Hi! Is there any chance you'll be able to publish a new NPM version today, @vperron?

@mtr mtr mentioned this pull request Jan 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant