wordpress-mobile / WordPress-Android

WordPress for Android
http://android.wordpress.org
GNU General Public License v2.0
2.99k stars 1.33k forks source link

feat: GutenbergKit Media Library support #21399

Closed dcalhoun closed 1 week ago

dcalhoun commented 3 weeks ago

Related

Description

Add support for attaching Media Library items to media blocks—Image, Gallery, Video, etc.

Testing Instructions

See https://github.com/wordpress-mobile/GutenbergKit/pull/36.

Regression Notes

  1. Potential unintended areas of impact Regressions in Gutenberg Mobile or Aztec media attachments.
  2. What I did to test those areas of impact (or what existing automated tests I relied on) Manually tested attaching media.
  3. What automated tests I added (or what prevented me from doing so) Deemed unnecessary for the experimental GutenbergKit editor.

PR Submission Checklist:

Testing Checklist (strike-out the not-applying and unnecessary ones):

dangermattic commented 3 weeks ago
1 Warning
:warning: PR is not assigned to a milestone.

Generated by :no_entry_sign: Danger

wpmobilebot commented 3 weeks ago

Project dependencies changes

The following changes in project dependencies were detected (configuration wordpressVanillaReleaseRuntimeClasspath):

list ``` Upgraded Dependencies org.wordpress.gutenbergkit:android:trunk-c62f3f03a4fdcd31263e9cc014f7a4c65c18131a, (changed from trunk-a58a46f3fbb892f311b562e3c122d7ef4ebbfe33) ```
tree ```diff +--- project :libs:editor -| \--- org.wordpress.gutenbergkit:android:trunk-a58a46f3fbb892f311b562e3c122d7ef4ebbfe33 +| \--- org.wordpress.gutenbergkit:android:trunk-c62f3f03a4fdcd31263e9cc014f7a4c65c18131a -\--- org.wordpress.gutenbergkit:android:trunk-a58a46f3fbb892f311b562e3c122d7ef4ebbfe33 (*) +\--- org.wordpress.gutenbergkit:android:trunk-c62f3f03a4fdcd31263e9cc014f7a4c65c18131a (*) ```
wpmobilebot commented 3 weeks ago
WordPress📲 You can test the changes from this Pull Request in WordPress by scanning the QR code below to install the corresponding build.
App NameWordPress WordPress
FlavorJalapeno
Build TypeDebug
Versionpr21399-caf348f
Commitcaf348f3614204a7fac243237c967eda62aae4ff
Direct Downloadwordpress-prototype-build-pr21399-caf348f.apk
Note: Google Login is not supported on these builds.
wpmobilebot commented 3 weeks ago
Jetpack📲 You can test the changes from this Pull Request in Jetpack by scanning the QR code below to install the corresponding build.
App NameJetpack Jetpack
FlavorJalapeno
Build TypeDebug
Versionpr21399-caf348f
Commitcaf348f3614204a7fac243237c967eda62aae4ff
Direct Downloadjetpack-prototype-build-pr21399-caf348f.apk
Note: Google Login is not supported on these builds.
codecov[bot] commented 1 week ago

Codecov Report

Attention: Patch coverage is 8.33333% with 11 lines in your changes missing coverage. Please review.

Project coverage is 39.50%. Comparing base (ae41f3f) to head (caf348f). Report is 99 commits behind head on trunk.

Files with missing lines Patch % Lines
...ress/android/ui/photopicker/MediaPickerLauncher.kt 0.00% 6 Missing :warning:
...rdpress/android/ui/mediapicker/MediaPickerSetup.kt 16.66% 5 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## trunk #21399 +/- ## ======================================= Coverage 39.50% 39.50% ======================================= Files 2120 2120 Lines 99438 99438 Branches 15282 15282 ======================================= Hits 39281 39281 Misses 56675 56675 Partials 3482 3482 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

nbradbury commented 1 week ago

Feedback provided in GutenbergKit repo here.

sonarcloud[bot] commented 1 week ago

Quality Gate Passed Quality Gate passed

Issues
3 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud