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

[RNMobile] Add support Reusable blocks #2627

Conversation

fluiddot
Copy link
Contributor

@fluiddot fluiddot commented Sep 16, 2020

Gutenberg PR -> WordPress/gutenberg#25265

Second part (adding Reusable blocks to the inserter menu):
Gutenberg PR -> WordPress/gutenberg#25383
Gutenberg-mobile PR -> #2644

Related to #2575

This PR is the first part of the issue Add Support to Insert Existing Reusable Blocks from the Editor.

The scope of this issue is limited to add Reusable blocks to the inserter menu, however due to the need of registering the Reusable block type (core/block), I also had to add support for presenting it in the editor.

The edition support will be added in this issue, so for now this PR only provides the preview mode of this type of block.

To test:
Check test steps in Gutenberg PR WordPress/gutenberg#25265

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.

Update develop branch from original repo
@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Sep 16, 2020

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

@fluiddot fluiddot marked this pull request as ready for review September 16, 2020 18:22
Update develop branch from original repo
@fluiddot fluiddot force-pushed the rnmobile/2575-support-reusable-blocks branch from 358be48 to 001737e Compare September 18, 2020 18:45
Update master branch from original repo.
@fluiddot fluiddot force-pushed the rnmobile/2575-support-reusable-blocks branch from 001737e to 30d95f6 Compare September 22, 2020 17:55
Update master branch from original repo
Update master branch from original repo
@fluiddot fluiddot force-pushed the rnmobile/2575-support-reusable-blocks branch 3 times, most recently from f593103 to 44fe0e4 Compare December 23, 2020 15:37
Update master branch from original repo
@fluiddot fluiddot force-pushed the rnmobile/2575-support-reusable-blocks branch from 44fe0e4 to 49e74cd Compare December 23, 2020 15:50
Update master branch from original repo
Update gutenberg ref

Update gutenberg ref
@fluiddot fluiddot force-pushed the rnmobile/2575-support-reusable-blocks branch from e2e58dd to c38df68 Compare December 30, 2020 18:18
Copy link
Contributor

@guarani guarani left a comment

Choose a reason for hiding this comment

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

Sorry for the delay getting to this, it slipped through.
Just a heads-up, this PR doesn't seem to be pointing to the latest commit on the related Gutenberg PR.
The tests fail only because this is a fork. I can push these commits to another branch, make a temporary PR and when CI passes, this PR will also automatically turn green.

@@ -14,6 +14,7 @@ Unreleased
* [***] Full-width and wide alignment support for Columns
* [***] New Block: File [https://github.com/wordpress-mobile/gutenberg-mobile/pull/2835]
* [**] Image block - Add link picker to the block settings and enhance link settings with auto-hide options [https://github.com/WordPress/gutenberg/pull/27292]
* [**] Add Support to Insert Existing Reusable Blocks from the Editor.
Copy link
Contributor

Choose a reason for hiding this comment

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

This can go in the Unreleased section above.

Copy link
Contributor Author

@fluiddot fluiddot Jan 6, 2021

Choose a reason for hiding this comment

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

I saw in a P2 post that the changes should be added in the CHANGELOG file in Gutenberg repository instead of here, right?
If that’s the case I’ll add this change in the Gutenberg PR so in the new PR for Gutenberg-mobile only the submodule reference update will be included.

Copy link
Contributor

Choose a reason for hiding this comment

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

True, I'd overlooked that, thanks for noticing!

@fluiddot
Copy link
Contributor Author

fluiddot commented Jan 6, 2021

Sorry for the delay getting to this, it slipped through.
Just a heads-up, this PR doesn't seem to be pointing to the latest commit on the related Gutenberg PR.
The tests fail only because this is a fork. I can push these commits to another branch, make a temporary PR and when CI passes, this PR will also automatically turn green.

Thanks for reviewing it @guarani!

I created a new PR directly in the gutenberg-mobile repository to pass all the checks:
#2953

@fluiddot
Copy link
Contributor Author

fluiddot commented Jan 6, 2021

I’m closing this PR because there’s a new one created directly in the Gutenberg-mobile repository:

#2953

@fluiddot fluiddot closed this Jan 6, 2021
@fluiddot fluiddot deleted the rnmobile/2575-support-reusable-blocks branch January 7, 2021 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants