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

Fix expected image name after upstream change #3749

Merged
merged 1 commit into from
Nov 14, 2019

Conversation

barklund
Copy link
Contributor

Summary

Fixes #3748

Checklist

  • My pull request is addressing an open issue (please create one otherwise).
  • My code is tested and passes existing tests.
  • My code follows the Engineering Guidelines (updates are often made to the guidelines, check it out periodically).

@barklund barklund added Bug Something isn't working AMP Stories labels Nov 14, 2019
@googlebot googlebot added the cla: yes Signed the Google CLA label Nov 14, 2019
@westonruter westonruter added this to the v1.4.1 milestone Nov 14, 2019
@westonruter westonruter added the Upstream Bug Requires an upstream change from WordPress, Gutenberg, or another dependency. label Nov 14, 2019
@westonruter westonruter merged commit ddd1a42 into develop Nov 14, 2019
@westonruter westonruter deleted the fix/3748-fix-e2e-tests branch November 14, 2019 18:32
@westonruter
Copy link
Member

Thank you 🙇

westonruter added a commit that referenced this pull request Nov 14, 2019
* tag '1.4.1': (26 commits)
  Bump 1.4.1
  Update screenshots for 1.4.1
  Fix expected image name after upstream change (#3749)
  Use length property instead of count() method on DOMNodeList (#3727)
  Improve display of validation errors for scripts (#3722)
  Conditionally run E2E tests (#3723)
  Tidy up validation error details (#3721)
  Bump 1.4.1-RC1
  Default to the homepage instead of fetching the first AMP compatible post to customize (#3715)
  Add missing space after sentence (#3720)
  Include text content of style element in validation error (#3717)
  Use bitwise operator.
  Check if element is not in top toolbar.
  Fix user select for meta date and author
  Allow right click for meta blocks
  Fix summarizing error sources both parent theme and child theme (#3709)
  Fix identifying sources for validation errors coming child themes (#3708)
  Fix failing E2E tests (#3707)
  Remove amp_validate query var from Validated URL 'View' row action (#3706)
  Escape instances of unescapeed output in AMP settings screen code (#3703)
  ...
westonruter added a commit that referenced this pull request Nov 15, 2019
…ve-duplicate-amp-scripts

* 'develop' of github.com:ampproject/amp-wp:
  Manually add copies of AMP scripts rather than using wp_print_scripts()
  Add duplicate scripts to head as well
  Add test for removing duplicate scripts
  Bump stable tag to 1.4.1
  Update screenshots for 1.4.1
  Fix expected image name after upstream change (#3749)
  Use length property instead of count() method on DOMNodeList (#3727)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working cla: yes Signed the Google CLA Upstream Bug Requires an upstream change from WordPress, Gutenberg, or another dependency.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

E2E tests are failing for background media uploads
3 participants