Skip to content
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

Fix error on android targeting S+ #9

Merged
merged 1 commit into from
Aug 23, 2024
Merged

Conversation

hans00
Copy link

@hans00 hans00 commented Feb 3, 2023

No description provided.

@flyskywhy
Copy link
Owner

Many IoT devices (include mine) with USB run in low version of Android e.g. 7.0, so I have no enviroment to test this PR.

Could you describe some info of your use case? (before I merge your PR 😝 )

@hans00
Copy link
Author

hans00 commented Feb 10, 2023

Our product is for consumer usage, which may include higher version.
I tested on Android 12 (Samsung Tablet S6).

Without this PR will throw error:

Connection Failed! Catch Error Message:com.brickslauncherbeta: Targeting S+ (version 31 and above) requires that one of FLAG IMMUTABLE or FLAG MUTABLE be specified when creating a PendingIntent. Strongly consider using FLAG IMMUTABLE, only use FLAG MUTABLE if some functionality depends on the Pendingintent being mutable, e.g. if it needs to be used with inline replies or bubbles.

@flyskywhy
Copy link
Owner

Thanks, and another question, If you are compiling for Android 12, your gradle version may greater than gradle-6.7.1, so how do you solve the compile problem as described in

// `Direct local .aar file dependencies are not supported when building an AAR.

@flyskywhy
Copy link
Owner

Thanks, and another question, If you are compiling for Android 12, your gradle version may greater than gradle-6.7.1, so how do you solve the compile problem as described in

// `Direct local .aar file dependencies are not supported when building an AAR.

So I can describe your solution into README.md

@flyskywhy flyskywhy merged commit ab874e0 into flyskywhy:master Aug 23, 2024
@flyskywhy
Copy link
Owner

Resolved in react-native-usb-serialport@3.0.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants