-
Notifications
You must be signed in to change notification settings - Fork 333
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 @ledger/react-native-hid for android tiramisu #7880
base: develop
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git βοΈ
2 Skipped Deployments
|
@nicolas-meilan is attempting to deploy a commit to the LedgerHQ Team on Vercel. A member of the Team first needs to authorize it. |
a0680d7
to
4421908
Compare
4421908
to
e96577b
Compare
114e6ca
to
edcd336
Compare
Hello @nicolas-meilan, first of all, thank you for your contribution! |
9e9e0d8
to
a27a8e3
Compare
Hello @nicolas-meilan, thank again for your continuous sync of your branch. Thanks again for your patience π |
@nicolas-meilan you can stop merging develop into your branch for now. As soon as it's reviewed, we will need one last merge/rebase and you should be good to go |
There as been no activity on this PR for the last 14 days. Please consider closing this PR. |
#7786
β Checklist
npx changeset
was attached.@ledger/react-native-hid
π Description
Android sdk 34 give the following error when use transporter from the lib @ledger/react-native-hid:
One of RECEIVER_EXPORTED or RECEIVER_NOT_EXPORTED should be specified when a receiver isn't being registered exclusively for system broadcasts
.To fix this bug is needed call
getReactApplicationContext()
with theContext.RECEIVER_NOT_EXPORTED
attribute.implementation that throw the error: https://github.com/nicolas-meilan/RungeCoin/blob/master/src/web3/wallet/wallet.ts#L210
β Context
π§ Checklist for the PR Reviewers