-
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
DynamicHookComponent does use the connected wallet #6
Comments
I am unsure if this is the best way to handle this so I'm skipping a PR at the moment in favor of discussion. I did work around using a hardcoded wallet and showing a balance before the wallet is connected by delaying access until after useCustomHook is not null examples/next-app-route/app/_components/dinamycHook.tsx Lines 27 to 32 in 314fab5
Then creating a new async function and calling it:
Which could instead be:
Unsure what the preferred example would be, but I think demonstrating how to interact with |
for
|
examples/next-app-route/app/_components/dinamycHook.tsx
Lines 14 to 17 in 314fab5
The dynamic hook is hardcoded to use eternl and does not show how to hook to a connected wallet.
Reproduction Steps
If there is no eternl wallet w/ approved Dapp it is easy to overlook in the developer console. The reproduction steps could be much shorter, but to demonstrate fully I have described the testing environment I used which made this quite apparent.
npm run dev
for next-app-route.Expected Behavior
The text was updated successfully, but these errors were encountered: