fix: Wallet disconnect should clear role context and reset to landing page state

[JamesVictor-O]

Problem

When a user disconnects their wallet (via the wallet dropdown), only the wallet state is cleared. The role context persists in localStorage, meaning:

1. User onboards as "business", role saved to localStorage
2. User disconnects wallet
3. User reconnects wallet — they're automatically routed to `/dashboard/business` without going through onboarding again
4. Or worse: a different user connects their wallet on the same browser and inherits the previous user's role

Files:

• `components/wallet/wallet-provider.tsx` (line 83-86) — `disconnect()` only clears wallet state
• `components/role/role-context.tsx` — role persists independently in localStorage

---

Fix

When the wallet disconnects, also clear the role:

Option A — Clear role in disconnect handler

Add `useRole` to the wallet dropdown and clear on disconnect:

`components/wallet/connect-wallet-button.tsx`:
```tsx
import { useRole } from "@/components/role/role-context";

// Inside the component:
const { setRole } = useRole();

// In the disconnect handler:
onClick={() => {
disconnect();
setRole(null); // Clear persisted role
setIsOpen(false);
}}
```

`components/wallet/wallet-chip.tsx`:
Same change in the dashboard wallet chip disconnect handler.

Option B — Add an `onDisconnect` callback to WalletProvider

Let consumers register cleanup callbacks that fire on disconnect.

---

Acceptance Criteria

• Disconnecting wallet clears the role from localStorage
• After disconnect, navigating to `/dashboard` redirects to `/dashboard/business` (default, not cached role)
• Reconnecting wallet and clicking "Start a campaign" opens fresh onboarding
• Works in both Navbar wallet button and dashboard wallet chip

[barry01-hash]

i would love to work on this issue

[barry01-hash]

i would love to work on this issue

[tydavid5]

hi team,
I'd like to work on this issue.

I have experience working with React/Next.js state management and localStorage persistence patterns, and I understand how stale client-side state can lead to incorrect routing and user context leakage.

For this fix, I will:

• Ensure role context is cleared alongside wallet disconnect in all disconnect entry points
• Update WalletProvider / wallet UI components so disconnect triggers role reset
• Remove persisted role from localStorage on disconnect
• Verify default routing behavior returns users to onboarding state
• Test both navbar wallet disconnect and dashboard wallet chip flows to ensure consistency across entry points

This will prevent role leakage between sessions and ensure correct onboarding flow after wallet reconnection.

[ekwe7]

I would love to work on this issue. I have experience building production ready applications and collaborating through GitHub workflows. I'll carefully analyze the current implementation, propose a reliable solution, and submit a well-tested PR. My focus is on clean code, maintainability, and timely delivery

[akinboyewaSamson]

Hi! I’m a full-stack developer with a strong blockchain background. I’d love to handle this issue and can deliver a clean, reliable solution. Kindly assign it to me.

[Ahmedscreativeverse]

i can resolve this issue and get soon

[sheyman546]

please i will love to work on this issue and i will submit within 24hr with a quality pr

[retkatmun]

Hi,

I’m a full-stack software engineer with experience building scalable web apps, backend systems,

I’d love to work on this issue. I’ve reviewed the requirements and have a clear implementation plan to deliver a clean, well-tested solution that meets all acceptance criteria and aligns with the project’s standards.

I can start immediately and provide timely updates throughout the implementation.

[grantfox-oss]

🦊 GrantFox — @Ahmedscreativeverse has been assigned to this issue as part of the Official Campaign campaign!

Next steps:

1. Open a Pull Request referencing this issue (e.g., Closes #81)
2. Your PR will be reviewed by the Ads-Bazaar maintainers

Good luck! Track your progress on GrantFox.