-
Notifications
You must be signed in to change notification settings - Fork 5.4k
feat: adding context for dapp swap #37980
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…ask-extension into dapp_swap_metrics
...es/confirmations/components/confirm/info/shared/edit-gas-fees-row/edit-gas-fees-row.test.tsx
Show resolved
Hide resolved
Builds ready [502cf4d]
UI Startup Metrics (1212 ± 97 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
|
ui/pages/confirmations/hooks/transactions/dapp-swap-comparison/useDappSwapComparisonInfo.ts
Show resolved
Hide resolved
Builds ready [f415c8e]
UI Startup Metrics (1291 ± 101 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
|
ui/pages/confirmations/hooks/transactions/dapp-swap-comparison/useDappSwapUSDValues.ts
Show resolved
Hide resolved
...confirmations/components/confirm/dapp-swap-comparison-banner/dapp-swap-comparison-banner.tsx
Show resolved
Hide resolved
Builds ready [8fae2a4]
UI Startup Metrics (1239 ± 96 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
|
Builds ready [d70ee9e]
UI Startup Metrics (1202 ± 100 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
|
chaitanyapotti
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm for web3auth code owner files
Builds ready [c87eecc]
UI Startup Metrics (1198 ± 101 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
Description
PR is doing 2 things:
Changelog
CHANGELOG entry:
Related issues
Fixes: MetaMask/metamask-mobile#22928
Manual testing steps
Screenshots/Recordings
NA
Pre-merge author checklist
Pre-merge reviewer checklist
Note
Adds a dedicated
DappSwapContextfor swap UI state, migrates confirmation components/hooks to it, and introduces adappSwapQaflag affecting comparison banner visibility and quote handling.DappSwapContextProvideranduseDappSwapContextforselectedQuoteandisQuotedSwapDisplayedInInfo.ConfirmContext.Confirmflow, tests, and Storybook stories withDappSwapContextProvider.DappSwapComparisonBanner,BaseTransactionInfo,EditGasFeesRow,SelectedGasFeeToken,TransactionData,EstimatedPointsSection,useResimulationAlert,useTransactionConfirm.useDappSwapActions,useDappSwapComparisonMetrics, reward text hook.dappSwapQaflag; banner displays if threshold met or QA enabled with a quote.selectedQuoteinto context; adjust display/CTA metrics events.useDappSwapUSDValuesskips native addresses when fetching fiat rates.useDappSwapContext; adjust expectations.DappSwapContextProvider.Written by Cursor Bugbot for commit c87eecc. This will update automatically on new commits. Configure here.