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

TransportRaceCondition in ledger-transport.js changed to TransportPendingOperation #950

Open
infoparth opened this issue Apr 24, 2024 · 7 comments

Comments

@infoparth
Copy link

Error: No matching export in "node_modules/@ledgerhq/errors/lib-es/index.js" for import "TransportRaceCondition"

node_modules/@solana/wallet-adapter-ledger/node_modules/@ledgerhq/hw-transport/lib-es/Transport.js:74:9:
  74 │ import { TransportRaceCondition, TransportError, StatusCodes, getAltStatusMessage, Transp...

which is causing the application to break on running "npm run dev",

because TransportRaceCondition has been updated to TransportPendingOperation, as mentioned here LedgerHQ/ledger-live@931d511

swapping TransportRaceCondition with TransportPendingOperation in the node modules worked for me, but it might break the application in production.

@jesulonimii
Copy link

Hey. noticed this issue also.
A work around I implemented was to modify the @ledgerhq/errors package from @ledgerhq/errors@6.16.4 to @ledgerhq/errors@6.16.3 in my package-lock.json. (and sha256 signature also)

So it uses the previous working version, pending the time this is resolved

@infoparth
Copy link
Author

Hey. noticed this issue also. A work around I implemented was to modify the @ledgerhq/errors package from @ledgerhq/errors@6.16.4 to @ledgerhq/errors@6.16.3 in my package-lock.json. (and sha256 signature also)

So it uses the previous working version, pending the time this is resolved

Hey @jesulonimii , thanks a lot for the help... it saved me

@wakechromium
Copy link

Hey. noticed this issue also. A work around I implemented was to modify the @ledgerhq/errors package from @ledgerhq/errors@6.16.4 to @ledgerhq/errors@6.16.3 in my package-lock.json. (and sha256 signature also)

So it uses the previous working version, pending the time this is resolved

Getting same error but enable to resolve.

@sunfkny
Copy link

sunfkny commented Apr 26, 2024

Hey. noticed this issue also. A work around I implemented was to modify the @ledgerhq/errors package from @ledgerhq/errors@6.16.4 to @ledgerhq/errors@6.16.3 in my package-lock.json. (and sha256 signature also)

So it uses the previous working version, pending the time this is resolved

@jesulonimii A better way is to add overrides in the package.json file of your project. #949

@shawalali07
Copy link

Hey. noticed this issue also. A work around I implemented was to modify the @ledgerhq/errors package from @ledgerhq/errors@6.16.4 to @ledgerhq/errors@6.16.3 in my package-lock.json. (and sha256 signature also)

So it uses the previous working version, pending the time this is resolved

Can you share the exact change here?

@mrsaifullah52
Copy link

after down grade the version it throws another error:

✘ [ERROR] No matching export in "node_modules/@ledgerhq/hw-transport/node_modules/@ledgerhq/errors/lib-es/index.js" for import "TransportPendingOperation"

    node_modules/@ledgerhq/hw-transport/lib-es/Transport.js:11:9:
      11 │ import { TransportPendingOperation, TransportError, StatusCodes, getAltStatusMessage, TransportStatusError, } from "@ledgerhq/errors";

@mcintyre94
Copy link
Collaborator

Can someone please provide source code that produces an error here?

I've updated the example app to include new LedgerWalletAdapter(), and I'm able to connect to a ledger wallet with no issues

I'd like to understand this error more before figuring out how we want to fix it here. Also note that TransportRaceCondition is not found in the source code of wallet-adapter, so I'm not sure yet where this error is coming from.

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

No branches or pull requests

7 participants