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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃И [Story performance] Disable animations on first page to 50% #35476

Merged

Conversation

mszylkowski
Copy link
Contributor

Set disabling animations experiment to 50%

@mszylkowski mszylkowski requested a review from newmuis July 30, 2021 17:59
@mszylkowski mszylkowski self-assigned this Jul 30, 2021
@mszylkowski mszylkowski added this to In progress in wg-stories Sprint via automation Jul 30, 2021
@amp-owners-bot
Copy link

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

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

@mszylkowski mszylkowski enabled auto-merge (squash) July 30, 2021 19:12
@gmajoulet
Copy link
Contributor

If the code is in production please use https://github.com/ampproject/amphtml/blob/main/build-system/global-configs/README.md#client-side-experiments-configjson for instant deployment of the experiment

@gmajoulet gmajoulet disabled auto-merge July 30, 2021 20:17
@mszylkowski mszylkowski enabled auto-merge (squash) August 2, 2021 16:04
@mszylkowski mszylkowski merged commit 228888c into ampproject:main Aug 2, 2021
wg-stories Sprint automation moved this from In progress to Done Aug 2, 2021
@ampprojectbot
Copy link
Member

This PR has been picked up by the CDN and will begin deploying shortly

@ampprojectbot
Copy link
Member

This experiment is now live and being served by the CDN

@mszylkowski mszylkowski deleted the experiment_disableanimations branch August 2, 2021 21:36
westonruter added a commit to westonruter/amphtml that referenced this pull request Aug 3, 2021
鈥ok-validation

* 'main' of github.com:ampproject/amphtml: (72 commits)
  build: run amp lint --fix to address import order of jixie (ampproject#35513)
  鉁 [amp-analytics] Add Custom Browser Event Tracker (ampproject#35193)
  babel: teach amp mode transformer about #core/mode (ampproject#35477)
  馃毊 Remove experiment `amp-consent-granular-consent` (ampproject#35508)
  鈾伙笍 Enable auto-sorting+grouping within src/ and 3p/ (ampproject#35454)
  馃悰  [amp-render] fix root-element stripping from amp-render with amp-bind (ampproject#35449)
  鉁 [Story interactive] Add Example Story for Detailed Results Component (ampproject#35450)
  馃悰 Fix error on bento example (ampproject#35490)
  馃悰 amp-story-grid-layer: Fix AMP invalidation error in documentation (ampproject#35503)
  馃悰 Fix code formatting (ampproject#35499)
  鉁 buildDom: add tests for amp-fit-text and amp-layout (ampproject#35494)
  鈾伙笍 Remove unused imports (ampproject#35435)
  鉁 amp-connatix-player: iframe domain based on a property (ampproject#35179)
  Updated document with use cases of remote config (ampproject#35496)
  AMP.goBack: update documentation (ampproject#29290)
  馃彈 Allow the bundle-size job to run even if the builds were skipped (ampproject#35492)
  build-system: improve terser/esbuild integration (ampproject#35466)
  馃И [Story performance] Disable animations on first page to 50% (ampproject#35476)
  馃摉 [Amp story] [Page attachments] Amp.dev Docs for New Page Attachment Features ampproject#34883 (ampproject#35338)
  馃殌 [Story interactive] Rewrite Image URL to Cached URL for Image Quizzes and Polls (ampproject#35375)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants