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

[Mobile]Remove unregistration of video block #15808

Merged
merged 2 commits into from
May 27, 2019

Conversation

pinarol
Copy link
Contributor

@pinarol pinarol commented May 24, 2019

Description

This PR makes video block publicly available

How has this been tested?

Tested with steps on gutenberg mobile PR: wordpress-mobile/gutenberg-mobile#1028

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows the accessibility standards.
  • My code has proper inline documentation.
  • I've included developer documentation if appropriate.

@pinarol pinarol added the Mobile App - i.e. Android or iOS Native mobile impl of the block editor. (Note: used in scripts, ping mobile folks to change) label May 24, 2019
@pinarol pinarol self-assigned this May 24, 2019
@pinarol pinarol requested a review from koke May 24, 2019 11:06
@hypest hypest merged commit de1960a into master May 27, 2019
@hypest hypest deleted the rnmobile/make-video-block-public branch May 27, 2019 08:15
@youknowriad youknowriad added this to the 5.8 (Gutenberg) milestone May 27, 2019
@hypest
Copy link
Contributor

hypest commented May 27, 2019

FYI, with Pinar AFK I took over updating the PR and merging it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Mobile App - i.e. Android or iOS Native mobile impl of the block editor. (Note: used in scripts, ping mobile folks to change)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants