Skip to content
This repository was archived by the owner on Jun 24, 2022. It is now read-only.

[1798 - Fee zero/null] add zero logic for fee #1800

Merged
merged 3 commits into from
Nov 9, 2021
Merged

Conversation

W3stside
Copy link
Contributor

@W3stside W3stside commented Nov 8, 2021

Summary

Closes #1798

Fee can be zero if gas is low, fixes this. Uses 🎉 None! if fee is zero

image

@W3stside W3stside requested review from a team November 8, 2021 11:19
@github-actions
Copy link
Contributor

github-actions bot commented Nov 8, 2021

  • 🔭 GP Swap: Gnosis Protocol v2 Swap UI

@elena-zh
Copy link

elena-zh commented Nov 8, 2021

Hey @W3stside , could you please explain which tokens to select for testing?
At least, I need to check how changes look like in tooltip and on the confirmation modal.

image

Co-authored-by: Leandro Boscariol <[email protected]>
@W3stside
Copy link
Contributor Author

W3stside commented Nov 9, 2021

Hey @W3stside , could you please explain which tokens to select for testing? At least, I need to check how changes look like in tooltip and on the confirmation modal.

image

Swap conf:
image

Swap page:
image

@W3stside W3stside requested a review from alfetopito November 9, 2021 14:23
@elena-zh
Copy link

elena-zh commented Nov 9, 2021

I'm approving this based on the provided screenshots as I can't get/simulate real data to test this.
Anyways, I will keep it in mind and retest it as soon as I face this situation.

@W3stside W3stside merged commit 53858f2 into develop Nov 9, 2021
W3stside added a commit that referenced this pull request Nov 9, 2021
* add zero logic for fee

* None! >> Free!

Co-authored-by: Leandro Boscariol <[email protected]>

* add gap to popover and Free when 0 fee

Co-authored-by: Leandro Boscariol <[email protected]>
@alfetopito alfetopito deleted the 1798/fee-zero branch November 9, 2021 19:44
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.

'Null' is displayed when a fee is 0
4 participants