Skip to content

Commit

Permalink
chore: upgrade ledgerhq depens, remove patch
Browse files Browse the repository at this point in the history
chore: upgrade ledgerhq depens, remove patch

The upstream issue that required the patch was corrected. These upgrades will allow for Clear Signing with new Ledger firmware versions

chore: upgrade @ledgerhq/hw-app-eth

chore: upgrade to new versions
  • Loading branch information
DanielSinclair committed Oct 1, 2024
1 parent 404a7ec commit 606f42b
Show file tree
Hide file tree
Showing 4 changed files with 176 additions and 91 deletions.
4 changes: 2 additions & 2 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -102,8 +102,8 @@
"@gorhom/bottom-sheet": "4.6.1",
"@json-rpc-tools/utils": "1.7.6",
"@lavamoat/preinstall-always-fail": "2.0.0",
"@ledgerhq/hw-app-eth": "6.29.9",
"@ledgerhq/react-native-hw-transport-ble": "6.27.7",
"@ledgerhq/hw-app-eth": "6.38.2",
"@ledgerhq/react-native-hw-transport-ble": "6.33.4",
"@metamask/eth-sig-util": "7.0.2",
"@notifee/react-native": "7.8.2",
"@rainbow-me/provider": "0.1.1",
Expand Down
15 changes: 0 additions & 15 deletions patches/@ledgerhq+react-native-hw-transport-ble+6.27.7.patch

This file was deleted.

4 changes: 2 additions & 2 deletions src/hooks/useLedgerImport.ts
Original file line number Diff line number Diff line change
Expand Up @@ -57,8 +57,8 @@ export function useLedgerImport({

const newObserver = TransportBLE.observeState({
// havnt seen complete or error fire yet but its in the docs so keeping for reporting purposes
complete: (e: any) => {
logger.debug('[useLedgerImport]: Observer complete', { e }, DebugContext.ledger);
complete: () => {
logger.debug('[useLedgerImport]: Observer complete', {}, DebugContext.ledger);
},
error: (e: any) => {
logger.debug('[useLedgerImport]: Observer error ', { e }, DebugContext.ledger);
Expand Down
Loading

0 comments on commit 606f42b

Please sign in to comment.