Skip to content

feat: implement manual wallet disconnect using useDisconnectWallet an…#2

Open
ombima56 wants to merge 1 commit into
mainfrom
fix/disconnect-toggle
Open

feat: implement manual wallet disconnect using useDisconnectWallet an…#2
ombima56 wants to merge 1 commit into
mainfrom
fix/disconnect-toggle

Conversation

@ombima56

Copy link
Copy Markdown
Collaborator

Description

This PR adds a manual wallet disconnect using useDisconnectWallet() from @mysten/dapp-kit.
Previously, the disconnect action depended on the internal behavior of ConnectButton.
With this update, users can now explicitly disconnect through our custom UI.

Changes

  • Added useDisconnectWallet() hook
  • Updated the Disconnect button to call disconnect()
  • Modal now closes immediately after disconnect
  • Removed reliance on ConnectButton’s internal disconnect logic

Why This Is Needed

This change:

  • Improves UX by giving us full control of the disconnect flow
  • Makes the UI more predictable and customizable
  • Prepares the ground for future enhancements (e.g., redirect, cleanup, confirmation dialog

@victorpaularony victorpaularony left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the change on the disconnect wallet feature is now up to date and functioning ok

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

Successfully merging this pull request may close these issues.

2 participants