Skip to content

Fix: ETH 'token' now only appears once in the swaps to and from dropdowns#10650

Merged
danjm merged 1 commit intodevelopfrom
fix-multiple-eth-swap-list
Mar 15, 2021
Merged

Fix: ETH 'token' now only appears once in the swaps to and from dropdowns#10650
danjm merged 1 commit intodevelopfrom
fix-multiple-eth-swap-list

Conversation

@danjm
Copy link
Contributor

@danjm danjm commented Mar 15, 2021

Previous to this PR users could see the ETH "token" three times in the "Swap from" and "Swap To" dropdowns of the the first screen of the swaps flow. This PR corrects this issue.

Before:

After:

@danjm danjm requested a review from a team as a code owner March 15, 2021 09:13
@danjm danjm requested a review from shanejonas March 15, 2021 09:13
@github-actions
Copy link
Contributor

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@danjm danjm requested a review from darkwing March 15, 2021 09:50
@danjm danjm force-pushed the fix-multiple-eth-swap-list branch from 8482568 to 9cb80c6 Compare March 15, 2021 12:53
@metamaskbot
Copy link
Collaborator

Builds ready [9cb80c6]
Page Load Metrics (695 ± 30 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint50836494
domContentLoaded5968316936330
load5978336956330
domInteractive5968316936330

Copy link
Contributor

@darkwing darkwing left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This feels much cleaner than previously!

@danjm danjm merged commit 60514c1 into develop Mar 15, 2021
@danjm danjm deleted the fix-multiple-eth-swap-list branch March 15, 2021 16:51
@github-actions github-actions bot locked and limited conversation to collaborators Mar 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants