Skip to content

🚀 [Story localization] Removed localization strings from js#38280

Merged
mszylkowski merged 6 commits intoampproject:mainfrom
mszylkowski:remove_localization
Jun 9, 2022
Merged

🚀 [Story localization] Removed localization strings from js#38280
mszylkowski merged 6 commits intoampproject:mainfrom
mszylkowski:remove_localization

Conversation

@mszylkowski
Copy link
Copy Markdown
Contributor

@mszylkowski mszylkowski commented Jun 8, 2022

Remove all localization strings to rely on the JS.

This removes the experiment flag because opting-out of this experiment cannot be done (the localizations strings are removed completely from the bundle).

@lgtm-com
Copy link
Copy Markdown

lgtm-com bot commented Jun 8, 2022

This pull request fixes 1 alert when merging 8709337 into 9cfa89f - view on LGTM.com

fixed alerts:

  • 1 for Syntax error

@mszylkowski mszylkowski requested a review from gmajoulet June 8, 2022 21:18
@mszylkowski mszylkowski self-assigned this Jun 8, 2022
@lgtm-com
Copy link
Copy Markdown

lgtm-com bot commented Jun 8, 2022

This pull request fixes 1 alert when merging 8382175 into b7954b6 - view on LGTM.com

fixed alerts:

  • 1 for Syntax error

@gmajoulet gmajoulet marked this pull request as ready for review June 8, 2022 21:40
@amp-owners-bot
Copy link
Copy Markdown

amp-owners-bot bot commented Jun 8, 2022

Hey @gmajoulet! These files were changed:

extensions/amp-story-shopping/0.1/test/test-amp-story-shopping-attachment.js
extensions/amp-story/1.0/amp-story.js
extensions/amp-story/1.0/test/test-amp-story.js

Hey @newmuis! These files were changed:

extensions/amp-story/1.0/amp-story.js
extensions/amp-story/1.0/test/test-amp-story.js

@gmajoulet
Copy link
Copy Markdown
Contributor

-9.3kB

200 (2)

@lgtm-com
Copy link
Copy Markdown

lgtm-com bot commented Jun 8, 2022

This pull request fixes 1 alert when merging 510b7c7 into b7954b6 - view on LGTM.com

fixed alerts:

  • 1 for Syntax error

@lgtm-com
Copy link
Copy Markdown

lgtm-com bot commented Jun 9, 2022

This pull request fixes 1 alert when merging 95f6f2e into 32e83d3 - view on LGTM.com

fixed alerts:

  • 1 for Syntax error

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.

3 participants