Skip to content

MediaFile - Add large and medium URLs#108

Merged
ParaskP7 merged 4 commits into
trunkfrom
add/large-medium-urls
Jul 20, 2022
Merged

MediaFile - Add large and medium URLs#108
ParaskP7 merged 4 commits into
trunkfrom
add/large-medium-urls

Conversation

@geriux

@geriux geriux commented Jun 30, 2022

Copy link
Copy Markdown

Related PRs:

Description

This PR adds the large and medium sizes for the MediaFile, needed for the editor's Image and Gallery block.

It also adds the getOptimalFileURL method to get the optimal image URL.

To test

Check the WordPress Android PR description.

@geriux geriux force-pushed the add/large-medium-urls branch from c21fa5d to 4df023a Compare July 6, 2022 14:07
@geriux geriux requested a review from derekblank July 12, 2022 13:26
@geriux geriux marked this pull request as ready for review July 12, 2022 13:26

@derekblank derekblank left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I was able to test and verify the steps following the Testing Instructions on WordPress/gutenberg#42178. LGTM! 🚀

Comment thread WordPressUtils/src/main/java/org/wordpress/android/util/helpers/MediaFile.java Outdated
@ParaskP7 ParaskP7 merged commit 68e2995 into trunk Jul 20, 2022
@ParaskP7 ParaskP7 deleted the add/large-medium-urls branch July 20, 2022 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants