fix: move setState calls inside async load function in useDashboard

[vaanni47]

Moves the setState (dispatch) calls inside an async load function within the useEffect in useDashboard.ts.

Previously, the effect used Promise chaining (.then/.catch) with dispatch calls outside an async context. This refactors it to use an inner async function load() so all state updates happen inside the async flow, which is cleaner and avoids potential race conditions.

Closes #76

[Copilot]

Pull request overview

Refactors useDashboard’s data-fetching effect to use an inner async load() function, keeping the dispatch flow (start/success/error) contained within a single async control path for readability and consistency.

Changes:

• Replaced Promise.all(...).then(...).catch(...) chaining with an inner async function using await + try/catch.
• Moved the FETCH_START dispatch into the async flow alongside the success/error dispatches.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[RITVIKKAMASETTY]

@vaanni47 can you write tests and run this npx prettier --write src/features/user/hooks/useDashboard.ts