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

Gallery - Media editing #2496

Merged
merged 10 commits into from
Aug 14, 2020
Merged

Gallery - Media editing #2496

merged 10 commits into from
Aug 14, 2020

Conversation

geriux
Copy link
Contributor

@geriux geriux commented Jul 21, 2020

Fixes #2389

Gutenberg PR -> WordPress/gutenberg#24088
WordPress iOS PR -> wordpress-mobile/WordPress-iOS#14498
WordPress Android PR -> wordpress-mobile/WordPress-Android#12497

Continuing adding media editing support to blocks, this PR adds it to Gallery block. For a more detailed description and testing plan check the Gutenberg PR description.

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](RELEASE-NOTES.txt) if necessary.

@geriux geriux force-pushed the feature/gallery-media-editing branch from 10f352d to ad8e1c6 Compare July 24, 2020 09:47
@geriux geriux requested a review from mkevins July 24, 2020 09:53
@geriux geriux marked this pull request as ready for review July 24, 2020 09:53
@ceyhun
Copy link
Contributor

ceyhun commented Aug 3, 2020

👋 @geriux ! We're getting ready to cut the 1.34 release branch, so I'm bumping this to the 1.35 milestone. If this needs to make the 1.35 release though, go ahead and change the milestone back and try to get this merged as soon as possible. If you're trying to get this into the release, please also let us know in Slack. 🙇

@ceyhun ceyhun modified the milestones: 1.34, 1.35 Aug 3, 2020
@geriux geriux force-pushed the feature/gallery-media-editing branch from ad8e1c6 to 6efaf03 Compare August 5, 2020 16:51
@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Aug 5, 2020

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

@geriux geriux force-pushed the feature/gallery-media-editing branch from 6efaf03 to 00a6b00 Compare August 6, 2020 07:26
@geriux geriux force-pushed the feature/gallery-media-editing branch from 00a6b00 to 4fd1280 Compare August 6, 2020 08:40
@geriux geriux force-pushed the feature/gallery-media-editing branch from 4fd1280 to 13ad7de Compare August 6, 2020 10:16
@geriux geriux force-pushed the feature/gallery-media-editing branch from 13ad7de to b5599e8 Compare August 13, 2020 10:48
@geriux geriux requested a review from lukewalczak August 14, 2020 08:10
@geriux geriux force-pushed the feature/gallery-media-editing branch from b5599e8 to 85b0b6e Compare August 14, 2020 08:12
@geriux
Copy link
Contributor Author

geriux commented Aug 14, 2020

Hey @lukewalczak 👋 since you already approved the Gutenberg PR, can you please approve this one as well? Thanks!

@geriux geriux merged commit 40cee3f into develop Aug 14, 2020
@geriux geriux deleted the feature/gallery-media-editing branch August 14, 2020 08:42
@cameronvoell cameronvoell mentioned this pull request Aug 18, 2020
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Gallery block: allow editing image
3 participants