Upgrade Gutenberg-Mobile Target SDK to 34
Task list
- [ ] Address error
java.lang.RuntimeException: Requested enabled DevSupportManager, but BridgeDevSupportManager class was not found or could not be createdby ~~patching React Native with this fix~~ upgrading React Native to version 0.71.13+ (https://github.com/WordPress/gutenberg/pull/57667) - [ ] Upgrade
targetSdkto 34 on Gutenberg (https://github.com/WordPress/gutenberg/pull/57918) - [ ] Upgrade
targetSdkto 34 onAztecEditor-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
compileSdkandtargetSdkto 34 in react-native-libraries-publisher (https://github.com/wordpress-mobile/react-native-libraries-publisher/pull/31) - [ ] Upgrade
compileSdkandtargetSdkto 34 inreact-native-slider(https://github.com/wordpress-mobile/react-native-slider/pull/15) - [ ] Upgrade
compileSdkandtargetSdkto 34 inreact-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
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 🙇 !
Thank you for confirming @fluiddot . I can use that as a checklist for understanding the process further. 🙏🏻
@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 🙇 !
@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.
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?
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.
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.
All tasks have been completed 🚀