Skip to content

Fixed Wallet Address Issue#122

Merged
SudiptaPaul-31 merged 3 commits intoCloudra-Labs:mainfrom
chiscookeke11:issue-#108
Jul 1, 2025
Merged

Fixed Wallet Address Issue#122
SudiptaPaul-31 merged 3 commits intoCloudra-Labs:mainfrom
chiscookeke11:issue-#108

Conversation

@chiscookeke11
Copy link
Contributor

Fix wallet address display and connection state management

Closes: #108

Problem

The wallet address was not properly displayed when connected and not properly cleared when disconnected, leading to inconsistent UI state and poor user experience.

Solution

Implemented proper wallet connection state management using Starknet React library to ensure wallet address is correctly displayed and managed throughout the application lifecycle.

Changes Made

1. Dependencies

  • Installed starknet-react library for robust wallet connection management

2. Context Setup

  • Added Starknet provider context to manage wallet state globally
  • Wrapped the main layout with the provider to ensure wallet state is available throughout the app

3. Component Updates

  • WalletModal: Updated to properly handle wallet connection/disconnection events
  • Toast Reports: Modified to show appropriate feedback for wallet state changes
  • Navbar: Updated to display wallet address when connected and clear it when disconnected

4. State Management

  • Wallet address now properly displays when wallet is connected
  • Wallet address is automatically removed when wallet is disconnected
  • Consistent state management across all components

Testing

  • Wallet connects and displays address correctly
  • Wallet disconnects and clears address properly
  • Toast notifications work for connection state changes
  • Navbar updates reflect current wallet state

@vercel
Copy link

vercel bot commented Jun 30, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
shadow-chat ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 1, 2025 3:28pm

@SudiptaPaul-31 SudiptaPaul-31 merged commit 76c1fb1 into Cloudra-Labs:main Jul 1, 2025
2 of 3 checks passed
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