Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: remove package from AndroidManifest.xml #2600

Merged
merged 1 commit into from
Jan 7, 2025

Conversation

jakex7
Copy link
Member

@jakex7 jakex7 commented Jan 7, 2025

Summary

Fixes #2599

Starting with v15.8.0, we support react-native@0.73+, which eliminates the need to worry about AGP versions below 7. Consequently, the package attribute in AndroidManifest.xml can be safely removed, along with the corresponding check in build.gradle.

Test Plan

Build the app on all supported versions

Compatibility

OS Implemented
Android

@jakex7 jakex7 requested a review from latekvo January 7, 2025 13:24
Copy link
Contributor

@latekvo latekvo left a comment

Choose a reason for hiding this comment

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

lgtm

@jakex7 jakex7 merged commit 6377be5 into main Jan 7, 2025
3 of 4 checks passed
@jakex7 jakex7 deleted the @jakex7/removeNamespaceInManifest branch January 7, 2025 13:44
jakex7 added a commit to jakex7/react-native-svg-ci that referenced this pull request Jan 9, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Setting the namespace via the package attribute in the source AndroidManifest.xml is no longer supported
2 participants