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

🌐 [Story subscription] Subscription localization async #38204

Merged

Conversation

mszylkowski
Copy link
Contributor

We should make the subscription service localize the template asychronously so we can lazy-load the localization strings.

@mszylkowski mszylkowski requested a review from ychsieh May 16, 2022 21:54
@mszylkowski mszylkowski self-assigned this May 16, 2022
@amp-owners-bot
Copy link

Hey @newmuis, @gmajoulet! These files were changed:

extensions/amp-story-subscriptions/0.1/amp-story-subscriptions.js
extensions/amp-story-subscriptions/0.1/test/test-amp-story-subscriptions.js

@ychsieh
Copy link
Contributor

ychsieh commented May 17, 2022

LGTM

@mszylkowski mszylkowski enabled auto-merge (squash) May 18, 2022 17:50
@mszylkowski mszylkowski merged commit fb3f61c into ampproject:main May 18, 2022
@mszylkowski mszylkowski deleted the subscription_localization_async branch May 18, 2022 18:01
westonruter added a commit that referenced this pull request May 20, 2022
…a-menu-images-validator-spec

* 'main' of github.com:ampproject/amphtml: (90 commits)
  🔥 [Story mediapool] Fix videos from mediapool with `noaudio` don't have audio when reused. (#38216)
  Hide progress bar on the control group of auto advance experiment (#38215)
  ✨ Add Bento Autocomplete Component (#37837)
  🌐 [Story subscription] Subscription localization async (#38204)
  Dable: add new optional parameter "channel" (#38199)
  ✨ [AMP Story Paywall] Enable developers to specify a custom subscriptions page index (#38175)
  SwG Release 0.1.22.217 (#38187)
  amp-script: implements new size limits for sandboxed scripts (#38185)
  🖍  Hide the system layer and progress bar in preview mode (#38163)
  added minItems (#38177)
  Prevent expandTemplate from ReDOSing (#38178)
  Change amp-story-subscriptions attribute name to reflect its flexibility (#38176)
  🐛 [Story Preview] Enable amp-video to play in preview mode (#38149)
  Added the possibility to get page count to story messaging api (#38170)
  SwG Release 0.1.22.216 (#38168)
  Allow @newmuis to update OWNERS files (#38169)
  ✨ Add Richaudience to RTC callout vendors (#38160)
  🚀  SunMedia: Update amp-ad (#38128)
  Remove option to deploy PR artifacts to a static website (#38152)
  added some vars and requests in gfksensic.json (#37722)
  ...
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