Clarify payout wording for coin-denominated bounties#413
Conversation
Greptile SummaryThis PR updates
Confidence Score: 5/5Safe to merge — the change is purely cosmetic, no payout values or data storage fields are affected. Both branches of No files require special attention for correctness; the Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A["formatBountyPayout(amountUsd, paymentCoin)"]
A --> B{paymentCoin set?}
B -- Yes --> C["'Pays $X.XX USD in COIN'"]
B -- No --> D["'Pays $X.XX USD'"]
C --> E["bounties/page.tsx list card\n(inline with DollarSign icon)"]
C --> F["bounties/[id]/page.tsx PriceBox\n(2xl bold amount slot)"]
C --> G["dashboard/bounties/page.tsx\ncreated bounties row (DollarSign icon)"]
C --> H["dashboard/bounties/page.tsx\nsubmissions row · …"]
D --> E
D --> F
D --> G
D --> H
Reviews (3): Last reviewed commit: "Remove redundant payout suffix from dash..." | Re-trigger Greptile |
|
Follow-up pushed to address the Greptile note about the dashboard submissions label. The branch now removes the trailing literal Latest head commit: |
Closes #401
Summary
formatBountyPayouthelper so coin-denominated payouts read asPays $X.XX USD in COIN$1.00 USD (paid in SOL)as1 SOLWhy this fix
The existing label is technically correct, but it front-loads the dollar amount and then appends the coin in parentheses, which makes
payout_usd: 1, payment_coin: "SOL"easy to misread as1 SOL.The new wording keeps the same underlying data and avoids implying that the numeric amount belongs to the coin itself.
Scope
src/lib/bounties.tsBecause the existing pages already call the shared formatter, no call-site changes were needed for the list/detail/dashboard views.
Validation
formatBountyPayoutis the shared helper used by the bounty browse page, bounty detail page, and dashboard submissions/created views