-
Notifications
You must be signed in to change notification settings - Fork 1.5k
fix(MM-66825): 16kb pagesize #9325
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
base: feat/MM-65147-upgrade-react-native-0773
Are you sure you want to change the base?
fix(MM-66825): 16kb pagesize #9325
Conversation
Coverage Comparison ReportGenerated on December 11, 2025 at 21:57:00 UTC |
| <View | ||
| className='ScheduledPostIndicator' | ||
| > |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure why there's a view with className. But with newer version of expo-image, it's throwing a typescript error.
| diff --git a/node_modules/expo-image/android/build.gradle b/node_modules/expo-image/android/build.gradle | ||
| index 11492a2..b6bb941 100644 | ||
| --- a/node_modules/expo-image/android/build.gradle | ||
| +++ b/node_modules/expo-image/android/build.gradle | ||
| @@ -44,7 +44,7 @@ dependencies { | ||
| kapt "com.github.bumptech.glide:compiler:${GLIDE_VERSION}" | ||
| api 'com.caverock:androidsvg-aar:1.4' | ||
|
|
||
| - implementation "com.github.penfeizhou.android.animation:glide-plugin:3.0.2" | ||
| + implementation("com.github.penfeizhou.android.animation:glide-plugin:3.0.3") | ||
| implementation "com.github.bumptech.glide:avif-integration:${GLIDE_VERSION}" | ||
|
|
||
| api 'com.github.bumptech.glide:okhttp3-integration:4.11.0' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Finally able to remove this now that we are updating to [email protected]
enahum
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's clarify before approving.. great work on this one
|
@DHaussermann the latest dependency update of expo-web-browser should fix the issue with https://mattermost.atlassian.net/browse/MM-66863 on Android. I have the app building right now. Should be ready in 20 mins or so. |
|
Hi @rahimrahman I have tested the Android fix for SAML and it is working. Do we know if anything else is needed to merge this? The only other issue found in smoke testing was the share extension did not work reliably on iOS and Android but when I compare to other builds this does not feel new. |
…825-16kb-pagesize
|
@rahimrahman given we are merging the base PR #9253 but no longer cherry picking should we remove the label here as well? do you need to pull |
I just did. About to build Android and to make sure everything build and do a quick test. |
Summary
Changes in this PR prepare the app to be in alignment with Google 16KB page size requirement.
Ticket Link
https://mattermost.atlassian.net/browse/MM-66825
Checklist
E2E iOS tests for PR.Device Information
This PR was tested on:
Google Pixel 6, Android 16
Google Pixel 8 Emulator, Android 16
Screenshots
Release Note