-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
🔥🐛[yahoonativeads-amp] code cleanup and bug fix #26325
Conversation
@dysfunc Could you rebase on the latest master. The failure seems irrelevant. |
@powerivq updated |
Master is now unbroken. Can you rebase it again? @dysfunc |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving the bundle-size check (temporary failure in the check runner, but this PR doesn't change anything that should increase the bundle-size of any of the tracked files)
* master: [yahoonativeads-amp] code cleanup and bug fix (ampproject#26325) rephrased reasoning for text node (ampproject#26393) Render video alt and title attributes in vertical rendering mdoe. (ampproject#26370) Revert "Update I2I & I2S to reflect new Open Source process (ampproject#25530)" (ampproject#26392) Skip amp story affiliate link test (ampproject#26386) Update I2I & I2S to reflect new Open Source process (ampproject#25530) ✨ Add support for `"intrisic"` layout for `<amp-script>` (ampproject#26369) 📖 Rename Dev Channel to Experimental Channel in docs and comments (ampproject#26255) ✨Add support for AMP Story Quiz Reaction API calls (ampproject#26242) clarified text node behavior (ampproject#26376) # Conflicts: # extensions/amp-script/amp-script.md # extensions/amp-timeago/amp-timeago.md
This PR performs some code cleanup by removing events hooks and resolves a minor bug.
onViewChange
hook