-
-
Notifications
You must be signed in to change notification settings - Fork 8
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
@walletconnect: The requested module 'pino' does not provide an export named 'levels' #344
Comments
After I tried to ignore this error I get some other error. What might be the cause? Would appreciate any help! Cc. @guybedford
|
I managed to make some progress on this in Main issues were:
With that it seems like it might be working now. Let me know if you need any further help. |
This is great, I now have the entire web3auth stack operational except for one library |
Thanks for checking so quickly and yea you are right it does not immediately show. Only when actually using new await CoinbaseAdapter().init() does it use the invalid constructor. Is this in scope?
|
Okay, so the above is actually an interop problem, which strictly speaking is a bug in this file: https://unpkg.com/@web3auth/coinbase-adapter@9.0.2/dist/lib.cjs/coinbaseAdapter.js In the above it does this: var CoinbaseWalletSDK = require('@coinbase/wallet-sdk');
// later on
new CoinbaseWalletSDK(...) when the file at "use strict";
Object.defineProperty(exports, "__esModule", { value: true });
exports.CoinbaseWalletSDK = void 0;
// Copyright (c) 2018-2024 Coinbase, Inc. <https://www.coinbase.com/>
const CoinbaseWalletSDK_1 = require("./CoinbaseWalletSDK");
exports.default = CoinbaseWalletSDK_1.CoinbaseWalletSDK;
var CoinbaseWalletSDK_2 = require("./CoinbaseWalletSDK");
Object.defineProperty(exports, "CoinbaseWalletSDK", { enumerable: true, get: function () { return CoinbaseWalletSDK_2.CoinbaseWalletSDK; } }); which therefore has the shape So the fix would be for the consumer to change their code to instead do: var { CoinbaseWalletSDK } = require('@coinbase/wallet-sdk'); or if they are concerned about interop issues to something like: var CoinbaseWalletSDK = require('@coinbase/wallet-sdk');
if (CoinbaseWalletSDK.CoinbaseWalletSDK)
CoinbaseWalletSDK = CoinbaseWalletSDK.CoinbaseWalletSDK; I would therefore advise posting an issue to the original repo about this further. If there is no desire to change on their side then we can potentially look into solving this through JSPM-side overrides, but usually we try to avoid that whenever possible. Let me know if you need any further help at all. |
And if they are transpiling ES modules, their transpilation isn't applying the |
Bug fixed in Web3Auth/web3auth-web#1967 |
Thanks for information, all good now. |
Was adding
@web3auth/wallet-connect-v2-adapter
but got"The requested module 'pino' does not provide an export named 'levels'"
The error seems to be originating from @WalletConnect package logger@2.1.2 dependency.
Is there something I can do to fix it myself?
The text was updated successfully, but these errors were encountered: