cowswap icon indicating copy to clipboard operation
cowswap copied to clipboard

Custom slippage around 5% reverts for some values to a lower value

Open MareenG opened this issue 3 years ago • 8 comments

Describe the bug When setting a custom slippage around 5% a wrong notification is shown for some values

How to Reproduce Steps to reproduce the behavior:

  1. Go to settings
  2. Set custom value between 5.00 - 5.1% in 0.01% steps
  3. See error for 5.02% and 5.06% - Enter a valid slippage percentage

Expected behavior Your transaction may be frontrun should be shown

Screenshots Bildschirmfoto 2021-04-08 um 11 02 38 Bildschirmfoto 2021-04-08 um 11 02 32 Bildschirmfoto 2021-04-08 um 11 02 47 ... Bildschirmfoto 2021-04-08 um 11 02 58

MareenG avatar Apr 08 '21 09:04 MareenG

Screenshot 2021-05-26 at 13 30 39 I see a different issue here: We shoudln't display the text "your order might be front run". This can't happen in GP, I guess we copied this from Uniswap but it doesn't apply in GP. I will create a new ticket for this.

annamsgeorge avatar May 26 '21 11:05 annamsgeorge

--> opened new issue for my point, suggesting to remove the error message: Wrong message about slippage and being front run #675

annamsgeorge avatar May 26 '21 11:05 annamsgeorge

The issue with 5.02 and 5.06 slippage percentages is still reproducible in v.0.17: slippage value is reverted to a previous one when enter these amounts.

elena-zh avatar Aug 02 '21 08:08 elena-zh

Not reproducible in v1.0.0 closing it

MareenG avatar Aug 12 '21 13:08 MareenG

I'm able to reproduce the issue in v.1.0.0 https://drive.google.com/file/d/1wrbyzc-cUpiDBNya0so3wJEDGejXFfYc/view

elena-zh avatar Aug 12 '21 13:08 elena-zh

True, it still reverts but doesn't show enter a valid slippage percentage anymore

MareenG avatar Aug 12 '21 14:08 MareenG

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If you think it shouldn't be closed, speak now or forever hold your peace.

stale[bot] avatar Mar 30 '22 12:03 stale[bot]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If you think it shouldn't be closed, speak now or forever hold your peace.

stale[bot] avatar Jun 20 '22 04:06 stale[bot]