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

Creating video Cover block in mobile app results in block validation error #28972

Open
dcalhoun opened this issue Feb 12, 2021 · 2 comments
Open
Labels
Mobile App - i.e. Android or iOS Native mobile impl of the block editor. (Note: used in scripts, ping mobile folks to change) [Priority] Low Used to indicate that the issue at hand isn't a top priority to address and can be handled later [Type] Bug An existing feature does not function as intended

Comments

@dcalhoun
Copy link
Member

Description

Creating a video Cover block within the WordPress mobile app results in block validation errors when re-editing the post.

Step-by-step reproduction instructions

  1. Download (or generate) build for the latest WordPress Android.
  2. In the Android app, create a post for a Simple WordPress.com site.
  3. Add a Cover block.
  4. Select a video for the Cover block background.
  5. Publish the post.
  6. Exit the post editor.
  7. Edit the same post in the mobile app OR edit the same post in the web editor.

Expected behaviour

No errors are displayed in the mobile app. No errors are displayed in the web console.

Actual behaviour

A React Native error is displayed regarding block validation. Opening the same post in the web editor displays a block validation error in the console.

Screenshots or screen recording (optional)

Error Screenshots

Web

video-cover-block-validation-error-web

Android

video-cover-block-validation-error-android

WordPress information

  • WordPress version: WordPress.com
  • Gutenberg version: 9.8.4
  • Are all plugins except Gutenberg deactivated? Yes
  • Are you using a default theme (e.g. Twenty Twenty-One)? No

Device information

  • Device: Samsung S20 5G FE
  • Operating system: Android 11
  • WordPress app version: 86fe5183b1cf12d9854f17de722d1978b443ddc9 of WordPress Android.
@dcalhoun dcalhoun 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 Feb 12, 2021
@hypest
Copy link
Contributor

hypest commented Feb 15, 2021

Thanks for raising the issue David! Any info whether this leads to a crash of the editor?

@dcalhoun
Copy link
Member Author

@hypest no, from my testing a crash does not occur in a release build.

Release build: The web editor displays a warning in the console.
Debug build: The RN warning is displayed in the mobile app and the web editor displays a warning in the console.

@mchowning mchowning added [Type] Bug An existing feature does not function as intended [Priority] Low Used to indicate that the issue at hand isn't a top priority to address and can be handled later labels Feb 16, 2021
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) [Priority] Low Used to indicate that the issue at hand isn't a top priority to address and can be handled later [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

No branches or pull requests

3 participants