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

Gutenberg - Fix Gallery block uploads when the editor is closed #15457

Merged
merged 4 commits into from
Dec 9, 2020

Conversation

geriux
Copy link
Contributor

@geriux geriux commented Dec 9, 2020

Fixes an issue found while sanity testing the 1.43.0 Gutenberg mobile release.

Fixes WordPress/gutenberg#27618

Due to this Gutenberg change that updated the values for the linkTo attribute, the Gallery processor stopped working correctly. This PR updates the values to the new ones:

  • mediafile
  • attachmentpost

It also takes into account the default value none to process the images without links.

To test:

  • Gallery block - Close post with an ongoing image upload - steps

Test the different Gallery link to options: None, Media File, and Attachment Page

PR submission checklist:

  • I have considered adding unit tests where possible.
  • I have considered adding accessibility improvements for my changes.
  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

Copy link

@test-case-reminder test-case-reminder bot left a comment

Choose a reason for hiding this comment

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

Here are some suggested test cases for this PR.

  • Gallery block - Close post with an ongoing image upload - steps
  • Simultaneous uploads - steps

If you think that suggestions should be improved please edit the configuration file here. You can also modify/add test-suites to be used in the configuration.

If you are a beginner in mobile platforms follow build instructions.

@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Dec 9, 2020

You can test the changes on this Pull Request by downloading it from AppCenter here with build number: 39013. IPA is available here. If you need access to this, you can ask a maintainer to add you.

@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Dec 9, 2020

You can trigger optional UI/connected tests for these changes by visiting CircleCI here.

@geriux geriux added this to the 16.4 milestone Dec 9, 2020
@geriux geriux marked this pull request as ready for review December 9, 2020 15:26
@antonis antonis self-requested a review December 9, 2020 15:36
Copy link

@antonis antonis left a comment

Choose a reason for hiding this comment

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

Nice work @geriux 👍
I confirm that the fix works as expected (tested on iPhone 11 / iOS 14 simulator).
The code also looks good :)

@guarani
Copy link
Contributor

guarani commented Dec 9, 2020

I added an issue WordPress/gutenberg#27618 for this PR to fix and update the PR description to reflect this.

Copy link
Contributor

@chipsnyder chipsnyder left a comment

Choose a reason for hiding this comment

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

Ran the sanity tests on this branch with the Gutenberg release and everything worked great. Thanks for taking care of this @geriux 🎉

@guarani guarani merged commit 55a0953 into develop Dec 9, 2020
@guarani guarani deleted the gutenberg/fix-gallery-block-processor branch December 9, 2020 18:17
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.

Fix Gallery block uploads when the editor is closed
4 participants