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

Enable reusable block only in WP.com sites #3490

Merged
merged 20 commits into from
May 21, 2021

Conversation

fluiddot
Copy link
Contributor

@fluiddot fluiddot commented May 12, 2021

guteberg PR: WordPress/gutenberg#31744
WordPress-iOS PR: wordpress-mobile/WordPress-iOS#16475
WordPress-Android PR: wordpress-mobile/WordPress-Android#14623

Why reusable block is disabled in self-hosted sites

Related issue: #3457

The main reason for disabling this block in self-hosted sites is related to the case of having a reusable block that contains a self-reference (a reference to the same reusable block). While in WP.com sites this is working properly, in self-hosted sites, fetching data from the endpoint that holds the data for reusable blocks (/wp-json/wp/v2/blocks) in this case, the site returns a bad response so we don't have data for displaying them.

NOTE: This is reproducible on self-hosted sites with and without Jetpack, although the error response is different.

To test:
Follow the testing instructions from gutenberg PR.

PR submission checklist:

  • I have considered adding unit tests where possible.
  • I have considered if this change warrants user-facing release notes more info and have added them to RELEASE-NOTES.txt if necessary.

@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented May 12, 2021

Wanna run full suite of Android and iOS UI tests? Click here and 'Approve' CI job!

@fluiddot fluiddot added this to the 1.53.0 (17.4) milestone May 12, 2021
@fluiddot fluiddot marked this pull request as ready for review May 12, 2021 11:57
@fluiddot fluiddot requested a review from jd-alexander May 12, 2021 11:58
fluiddot added 10 commits May 13, 2021 19:30
# Conflicts:
#	bundle/ios/App.js
#	bundle/ios/App.js.map
#	bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/en-gb.json
#	bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/nl.json
#	bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/ro.json
#	bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/sv.json
# Conflicts:
#	bundle/ios/App.js
#	bundle/ios/App.js.map
@ceyhun
Copy link
Contributor

ceyhun commented May 14, 2021

Hey there @fluiddot 👋 We'll be cutting the 1.53.0 release some time soon today. I'm bumping this PR to 1.54.0 milestone, please let us know if it needs to make it into 1.53.0 instead.

Copy link
Contributor

@jd-alexander jd-alexander left a comment

Choose a reason for hiding this comment

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

fluiddot added 4 commits May 20, 2021 16:39
# Conflicts:
#	bundle/android/strings.xml
#	bundle/ios/App.js
#	bundle/ios/App.js.map
#	bundle/ios/GutenbergNativeTranslations.swift
#	bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/en-za.json
#	bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/it.json
#	bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/sq.json
@fluiddot fluiddot merged commit 8d406bf into develop May 21, 2021
@fluiddot fluiddot deleted the add/reusable-block-capability branch May 21, 2021 11:39
@jhnstn jhnstn mentioned this pull request May 28, 2021
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants