QA Sync: Inbox as Default Landing Page, Dashboard Temporarily Hidden, and Grid Query Stability Fixes
Executive summary
This release promotes a batch of accumulated changes from the production line into the QA environment. The most user-visible changes make the Mails inbox the default landing page after login and temporarily hide the in-progress Dashboard from the navigation, alongside internal stability fixes to grid column data loading and the addition of automated code-review tooling.
Why this was needed
The Dashboard feature is still under development and was not ready to be exposed to users, so it needed to be hidden while keeping the rest of the application fully usable. With the Dashboard removed as the home page, users logging in were being routed to an empty landing area instead of a working screen. Separately, the grid column-value data loading used a deprecated configuration option that needed updating to remain reliable.
Client / user impact
- After logging in, users now land directly on the Mails inbox instead of the (hidden) Dashboard, giving them an immediately useful starting screen.
- The Dashboard menu item no longer appears in the left navigation, avoiding confusion from an unfinished feature.
- Column filter/value dropdowns in data grids load more dependably due to corrected caching behavior.
- No change to mail processing, document handling, or user data.
Technical scope
- Login redirect (
src/providers/auth.provider.tsx): post-login navigation changed from/to/inbox. - Navigation (
src/constants/index.tsx): the Dashboard nav entry is commented out and its now-unusedLayoutDashboardicon import removed; minor whitespace/formatting cleanup. - Grid column query (
src/tanstack/queries/grid-columns.queries.ts): TanStack Query v5 migration — renamedcacheTimetogcTimeand removed the deprecatedonErrorconsole-logging callback. - CI tooling: added
.github/workflows/claude.ymlandclaude-code-review.ymlfor automated, on-demand AI code review (no runtime/app impact). - Dev config:
.claude/settings.local.jsonallowlists thepnpm addcommand (local developer tooling only). - Reflects merged work including #324 (hide dashboard), #325 (login redirect + @mantine/form dependency), #335 (mail update logic), and #339 (grid column query fixes).
Risk & mitigation
Low risk. Changes are small, mostly configuration and a navigation/redirect adjustment. The Dashboard route still exists in code (only the menu link and home redirect changed), so it can be restored quickly. Main watch item is ensuring no flow still assumes / as the post-login destination. Mitigation: verify the login-to-inbox flow and confirm no broken links point to the hidden Dashboard.
QA validation focus
- Log in as Internal Admin and Client Admin and confirm you land on /inbox, not a blank or error page.
- Confirm the Dashboard item is absent from the left navigation for all roles.
- Manually navigating to
/should not produce an error or trap the user. - Open data grids (e.g. inbox/document lists) and exercise column filter dropdowns; confirm values load and refresh correctly with no console errors.
- Smoke-test core mail listing and detail views to confirm the sync introduced no regressions.