Skip to content
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.

Sign up for GitHub

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

Jump to bottom

feat: improve review page with route tracker, added new fee card and improved route card #268

Merged
merged 5 commits into from
Jul 11, 2024
Merged

feat: improve review page with route tracker, added new fee card and improved route card #268

merged 5 commits into from
Jul 11, 2024

Conversation

chybisov
Copy link
Member

@chybisov chybisov commented Jul 10, 2024 •
edited
Loading

  1. Introduced a fee card displaying the minimum amount received and token rates.
  2. Route cards now display token rates with subscript representation for long decimals, which can be reversed on click.
  3. Added a route refresh button to the review page, and the route automatically updates every 1 minute.
  4. Price impact is now shown next to the dollar amount of the destination token.
  5. Amounts less than $0.01 are now shown as <$0.01 instead of $0.00 or $0.
  6. The title "LI.FI Dex Aggregator via LI.FI" step tool is now displayed as "LI.FI Dex Aggregator".
  7. Fees are now correctly sorted within tooltips.
image image

@chybisov chybisov requested a review from DNR500 July 10, 2024 09:37
@github-actions GitHub Actions
Copy link

Hey! This is your new endopint: https://5b76d9d8.widget-refactorfe.pages.dev

DNR500
DNR500 approved these changes Jul 10, 2024
View reviewed changes
Copy link
Contributor

@DNR500 DNR500 left a comment

Choose a reason for hiding this comment

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

Looks good

@github-actions GitHub Actions
Copy link

Hey! This is your new endopint: https://3e285d99.widget-refactorfe.pages.dev

@github-actions GitHub Actions
Copy link

Hey! This is your new endopint: https://d59c8097.widget-refactorfe.pages.dev

@chybisov chybisov merged commit 258c5d3 into main Jul 11, 2024
1 check passed
@chybisov chybisov deleted the refactor-fees branch July 11, 2024 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DNR500 DNR500 DNR500 approved these changes

Assignees
No one assigned
Labels
testing
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@chybisov @DNR500