interface
interface copied to clipboard
fix: Use current network name in Gas Breakdown tooltip
Description
Fixes the GasBreakdownTooltip
to use the current network name instead of the hardcoded "Ethereum network" text
Screen capture
Before
After
Test plan
Reproducing the error
- Switch to any non-EthereumMainnet network
- Go to Swaps
- Select "to" and "from" tokens
- Wait for the the gas estimate icon to display
- Hover of the gas estimate icon to see the GasBreakdownTooltip
- Ensure that the network name matches the current network
QA (ie manual testing)
- [ ] N/A
Devices
Automated testing
- [ ] Unit test
- [ ] Integration/E2E test
The latest updates on your projects. Learn more about Vercel for Git βοΈ
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
interface | β Ready (Inspect) | Visit Preview | π¬ Add feedback | Oct 12, 2023 1:51am |
Codecov Report
Merging #7292 (75f12b3) into main (5fee3c6) will decrease coverage by
2.43%
. The diff coverage is100.00%
.
:exclamation: Current head 75f12b3 differs from pull request most recent head 1bde62e. Consider uploading reports for the commit 1bde62e to get more accurate results
Flag | Coverage Ξ | |
---|---|---|
cloud-tests | 87.27% <ΓΈ> (+3.66%) |
:arrow_up: |
unit-tests | 40.28% <100.00%> (-2.54%) |
:arrow_down: |
Flags with carried forward coverage won't be shown. Click here to find out more.
this is a duplicate of #6979 but we'll probably merge whichever one updates first
@darkwing can you rebase and fix conflicts?
Updated as requested!