Skip to content

Limit Stories experience to WP 5.3 & Gutenberg 7.1.0#4217

Merged
westonruter merged 5 commits intodevelopfrom
add/4200-disable-amp-stories
Feb 8, 2020
Merged

Limit Stories experience to WP 5.3 & Gutenberg 7.1.0#4217
westonruter merged 5 commits intodevelopfrom
add/4200-disable-amp-stories

Conversation

@pierlon
Copy link
Copy Markdown
Contributor

@pierlon pierlon commented Feb 3, 2020

Summary

Disables Stories experience if Gutenberg > v7.1.0 or WordPress > v5.3. The Stories editor is broken in Gutenberg v7.2.0, and Wordpress 5.4 ( which will include an incompatible version of the block editor).

Fixes #4200.

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).

@googlebot googlebot added the cla: yes Signed the Google CLA label Feb 3, 2020
Co-Authored-By: Pascal Birchler <pascalb@google.com>
@pierlon pierlon requested a review from swissspidy February 8, 2020 05:47
.dev-lib Outdated
echo -n "Installing Gutenberg..."
gutenberg_plugin_svn_url=https://plugins.svn.wordpress.org/gutenberg/trunk/
echo -n "Installing Gutenberg 7.1.0..."
gutenberg_plugin_svn_url=http://plugins.svn.wordpress.org/gutenberg/tags/7.1.0/
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, why the change from https to http?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Revised in 983efd4

@westonruter westonruter added this to the v1.4.3 milestone Feb 8, 2020
@westonruter
Copy link
Copy Markdown
Member

Once the build passes I'll squash merge this and cherry-pick into the 1.4 branch.

@westonruter westonruter merged commit 224c5c2 into develop Feb 8, 2020
@westonruter westonruter deleted the add/4200-disable-amp-stories branch February 8, 2020 17:31
westonruter added a commit that referenced this pull request Feb 8, 2020
* Limit Stories experience to WP 5.3 & Gutenberg 7.1.0

* Fix failing E2E tests

* Fix doc block comment

Co-Authored-By: Pascal Birchler <pascalb@google.com>

* Return false if GUTENBERG_DEVELOPMENT_MODE is set

* Restore HTTPS protocol for gutenberg_plugin_svn_url

Co-authored-by: Pascal Birchler <pascal.birchler@gmail.com>
Co-authored-by: Weston Ruter <westonruter@google.com>
westonruter added a commit that referenced this pull request Feb 13, 2020
* tag '1.4.3': (22 commits)
  Update readme and screenshots for Stories removal (#4259)
  Open story export instructions in a new window (#4258)
  Bump version to 1.4.3-RC1
  Hide Stories options and add deprecation notice (#4219)
  Fix malformed conversion of relative action URLs for forms (#4250)
  Limit Stories experience to WP 5.3 & Gutenberg 7.1.0 (#4217)
  Prevent errors in admin bar filters from non-array arguments (#4207)
  Update @wordpress/e2e-test-utils dependency
  Revert update of mustache/mustache dependency
  Update composer.lock
  Update WP CLI to 2.4.0
  For WordPress.tv embed, Use an oembed filter instead of block filter (#4164)
  Update readme to add FAQs section (#4159)
  Apply workaround to fix test__multiple_valid_image_files (#4034)
  Ignore Story editor tests (#4043)
  Update amp-video embed regex pattern to include other Vimeo URL formats (#4051)
  Update amp-instagram embed regex (#4053)
  Update wp-dev-lib package (#4029)
  Fix conversion of forms with relative action URLs (#4003)
  Improve release instructions (#3995)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Signed the Google CLA

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Disable AMP Stories experience when recent Gutenberg is active

4 participants