Remove USD price displays across wallet and bridge - #518
Merged
Conversation
We operate OTC now, so the apps show TORUS amounts only: - Wallet: the TORUS/USD currency-swap input becomes a plain TORUS amount input (fraction buttons kept); the review dialog loses its "Amount (USD)" / "Network Fee (USD)" rows; transaction history stops annotating amounts with USD; the UsdPriceProvider context, conversion helpers, and stored usdPrice tx metadata are deleted. - Bridge: fast-bridge balance lines drop their USD parentheticals and the price query; the test mock follows. - Shared UI: the APR ticker loses its USD PRICE segment and price fetch, TorusToUSD is deleted, and the agent card sheds its dead usdValue prop. Chain gas prices (bridge-v2 withdraw plumbing) are untouched - those are execution parameters, not fiat display. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
We operate OTC now — the apps display TORUS amounts only, no fiat price anywhere.
UsdPriceProvider, the conversion helpers, and the storedusdPricetx metadata are deleted.TorusToUSDdeleted; agent card sheds a deadusdValueprop.Untouched on purpose: bridge-v2's chain
gasPriceplumbing (execution parameter, not fiat display) and the portal'suse-weekly-usdhook (portal routes are taken down).Net −344 lines. Verified: typecheck/lint/test green on all four packages (353 bridge tests pass), and a local wallet run shows the APR bar without the price segment and the send form as a single TORUS input.
🤖 Generated with Claude Code