Skip to content

fix: navbar loading skeleton and multi-role dashboard links (#48) - #150

Open
rafaio1 wants to merge 3 commits into
MergeFi:mainfrom
rafaio1:fix/48-navbar-loading-multrole
Open

rafaio1 wants to merge 3 commits into
MergeFi:mainfrom
rafaio1:fix/48-navbar-loading-multrole

Conversation

@rafaio1

@rafaio1 rafaio1 commented Aug 20, 2026

Copy link
Copy Markdown

Fixes #48.

  • Render stable skeleton placeholder during auth loading to prevent flash of logged-out state on fresh page load with persisted session.
  • Filter dashboard dropdown links by user.roles array, supporting multi-role users (e.g. maintainer+sponsor sees both links).
  • Hide Dashboards dropdown entirely when user has no assigned roles.
  • TypeScript compiles cleanly (npx tsc --noEmit passes).

root added 3 commits August 20, 2026 05:11
Fixes MergeFi#52.

ConnectPanel assumed GitHub auth and wallet connection happened in a
single linear session, failing to correctly reflect or handle partially
completed states (e.g. wallet connected locally but not linked to an
authenticated account, or an account linked to a different wallet than
the one currently active in Freighter).

Updated the wallet connection section to explicitly surface the linkage
state. It now distinguishes between a locally connected wallet, a
wallet durably linked to the authenticated GitHub account, and cases
where the local Freighter account diverges from the server-side linked
address, providing clear UI feedback for each matrix state.
…i#70)

- Pass raw numeric values with explicit format props on HomePage and
  ReputationPage instead of pre-formatted strings, restoring tooltip
  precision and zero-state detection.
- Add fetchReputationWithStatus to distinguish live fetch success from
  mock fallback; wire status prop into ReputationPage StatCards.
- Keep Avg. payout time as pre-formatted string with explicit comment
  (duration doesn't fit existing format variants).
- Render stable skeleton placeholder during auth loading to prevent
  flash of logged-out state on fresh page load with persisted session.
- Filter dashboard dropdown links by user.roles array, supporting
  multi-role users (e.g. maintainer+sponsor sees both links).
- Hide Dashboards dropdown entirely when user has no assigned roles.
@vercel

vercel Bot commented Aug 20, 2026

Copy link
Copy Markdown

Someone is attempting to deploy a commit to the chonilius' projects Team on Vercel.

A member of the Team first needs to authorize it.

This branch has not been deployed

No deployments
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.

Fix Navbar loading-state flashes and multi-role rendering logic

1 participant