Skip to content

fix: use StatCard numeric format and surface live/mock status (#70) - #149

Open
rafaio1 wants to merge 2 commits into
MergeFi:mainfrom
rafaio1:fix/70-statcard-bypass
Open

rafaio1 wants to merge 2 commits into
MergeFi:mainfrom
rafaio1:fix/70-statcard-bypass

Conversation

@rafaio1

@rafaio1 rafaio1 commented Aug 20, 2026

Copy link
Copy Markdown

Fixes #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).
  • TypeScript compiles cleanly (npx tsc --noEmit passes).

root added 2 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).
@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.

HomePage and ReputationPage bypass StatCard's status/format system, silently discarding precision and the zero-state

1 participant