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

🌸 Cherry-pick request for #34589 into #34641, #34705 #34740

Closed
2 of 7 tasks
erwinmombay opened this issue Jun 7, 2021 · 2 comments
Closed
2 of 7 tasks

🌸 Cherry-pick request for #34589 into #34641, #34705 #34740

erwinmombay opened this issue Jun 7, 2021 · 2 comments
Assignees
Labels

Comments

@erwinmombay
Copy link
Member

erwinmombay commented Jun 7, 2021

Issue (P0 Bug)

ampproject/error-reporting#90

Pull Request(s)

#34589

Release Tracker(s)

#34641, #34705

Channels

Beta / Experimental, Stable

Justification

errors related amp-img-gallery have increased.

see Error: b.signals is not a function errors. we should patch prod and experimental as soon as possible.
Also see ampproject/error-reporting#90

Verification Steps

Uncaught (in promise) TypeError: b.signals is not a function
    at hb.push.f.g.layoutCallback (amp-image-viewer-0.1.js:49)
    at v0.js:276
    at v0.js:6
    at new Promise (<anonymous>)
    at qa (v0.js:6)
    at HTMLElement.Ui.b.layoutCallback (v0.js:276)
    at v0.js:229
    at fm (v0.js:478)
    at bm.g.zh (v0.js:477)

Summary

A revert that was necessary to remove the img support did not land into the latest stable which is currently causing errors to increase in stable/beta/experimental

Impact

errors related to amp-image-gallery

Action Items

N/A

Cherry-Pick Progress

To be updated by @ampproject/release-on-duty as each stage is completed.

  • Cherry-pick request created
  • Cherry-pick request approved
  • New version released to Beta / Experimental channels
  • New version released to Stable channel
  • New version released to LTS channel
  • Release tracker updated
  • Cherry-pick completed

Notifications

/cc @ampproject/release-on-duty @ampproject/wg-approvers @ampproject/cherry-pick-approvers

@erwinmombay erwinmombay changed the title 🌸 Cherry-pick request for #ISSUE into #RELEASE 🌸 Cherry-pick request for #34589 into #34641, #34705 Jun 7, 2021
@erwinmombay erwinmombay self-assigned this Jun 7, 2021
@kristoferbaxter
Copy link
Contributor

Cherry pick approved. Let's also (and this is on me) ensure we add additional tests coverage for the revert of this revert.

Confusing, but important.

@erwinmombay
Copy link
Member Author

we promoted the latest experimental branch which already had the fix to stable to fix the lightbox-gallery issue.
went ahead and stopped the cherry pick process.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants