gutenberg-mobile icon indicating copy to clipboard operation
gutenberg-mobile copied to clipboard

Upgrade Gutenberg-Mobile Target SDK to 34

Open notandyvee opened this issue 1 year ago • 7 comments

Task list

  • [ ] Address error java.lang.RuntimeException: Requested enabled DevSupportManager, but BridgeDevSupportManager class was not found or could not be created by ~~patching React Native with this fix~~ upgrading React Native to version 0.71.13+ (https://github.com/WordPress/gutenberg/pull/57667)
  • [ ] Upgrade targetSdk to 34 on Gutenberg (https://github.com/WordPress/gutenberg/pull/57918)
  • [ ] Upgrade targetSdk to 34 on AztecEditor-Android (https://github.com/wordpress-mobile/AztecEditor-Android/issues/1072)
  • [ ] Bump the AztecEditor-Android version on Gutenberg
  • [ ] Bump the AztecEditor-Android version on WPAndroid
  • [ ] Upgrade compileSdk and targetSdk to 34 in react-native-libraries-publisher (https://github.com/wordpress-mobile/react-native-libraries-publisher/pull/31)
  • [ ] Upgrade compileSdk and targetSdk to 34 in react-native-slider (https://github.com/wordpress-mobile/react-native-slider/pull/15)
  • [ ] Upgrade compileSdk and targetSdk to 34 in react-native-video (https://github.com/wordpress-mobile/react-native-video/pull/23)
  • [ ] Publish a new version
  • [ ] Update WPAndroid

Note: I'm not certain as to the full process involved in doing that here. Will compare with the Android 13 issue - https://github.com/wordpress-mobile/gutenberg-mobile/issues/5556

notandyvee avatar Jan 10 '24 21:01 notandyvee

Hey @notandyvee 👋 , the process to update the target SDK will be similar to the one documented in https://github.com/wordpress-mobile/gutenberg-mobile/issues/5556. The main difference is that in this case, we'll need to update fewer third-party libraries. I took the liberty to update the task list in the issue's description. Thanks 🙇 !

fluiddot avatar Jan 11 '24 09:01 fluiddot

Thank you for confirming @fluiddot . I can use that as a checklist for understanding the process further. 🙏🏻

notandyvee avatar Jan 11 '24 14:01 notandyvee

@notandyvee I couldn't find a branch in WordPress-Android related to the target SDK upgrade. I'd like to test the React Native patch to address the error java.lang.RuntimeException: Requested enabled DevSupportManager, but BridgeDevSupportManager class was not found or could not be created. Let me know if you could share the branch, or if it's not created yet, ping me when it's ready. Thanks 🙇 !

fluiddot avatar Jan 15 '24 17:01 fluiddot

@notandyvee I have prepared the different PRs needed for this tracking issue. The only one remaining is updating the Aztec version, however, we'd first need to tackle https://github.com/wordpress-mobile/AztecEditor-Android/issues/1072. Let me know if you are planning to work on it, in the meantime, I'm going to pause my work on this issue. Thanks 🙇 !

⚠️ I've tested WP-Android app after updating targetSdk in an emulator with Android 14, and noticed that the app crashes when uploading media. I thought this was related to the editor, but it can also be reproduced when uploading from the Media screen. Sharing this here so you are aware of this issue when working in https://github.com/wordpress-mobile/WordPress-Android/issues/19917.

fluiddot avatar Jan 17 '24 15:01 fluiddot

Firstly, thank you @fluiddot for working on this so quickly. Extremely helpful 🙏🏻 .

I've tested WP-Android app after updating targetSdk in an emulator with Android 14, and noticed that the app crashes when uploading media.

Known issue. I have a fix locally. It's a security issue required by the target sdk update. Nothing to do with gutenberg. I'll put up a PR later today so we can use it for testing and for visibility.

The only one remaining is updating the Aztec version, however, we'd first need to tackle https://github.com/wordpress-mobile/AztecEditor-Android/issues/1072.

I can attempt to tackle that. How are gutenberg and aztec related?

notandyvee avatar Jan 17 '24 17:01 notandyvee

The only one remaining is updating the Aztec version, however, we'd first need to tackle wordpress-mobile/AztecEditor-Android#1072.

I can attempt to tackle that. How are gutenberg and aztec related?

Gutenberg Mobile uses Aztec for rendering rich text content. Most of the text rendered within the editor uses it.

fluiddot avatar Jan 18 '24 10:01 fluiddot

Apologies for the delay @fluiddot . The PR with the updates to avoid image crashing are here: https://github.com/wordpress-mobile/WordPress-Android/pull/19979. I will tackle Aztec next.

notandyvee avatar Jan 18 '24 22:01 notandyvee

All tasks have been completed 🚀

geriux avatar May 20 '24 09:05 geriux