gnosis / cowswap

🐮 CowSwap: First Gnosis Protocol v2 UI
https://gnosis.io
GNU General Public License v3.0
113 stars 55 forks source link

[presign] Add onchain cancelation of orders #1407

Open anxolin opened 3 years ago

anxolin commented 3 years ago

Description This issue is to implement onchain cancelation of orders.

Something to decide is:

Context

When implementing presign, we decided to remove temporarilly cancelation of orders, since we can't sign the offchain order as EOA do.

This issue, is just to reiterate and allow SC to cancel orders.

cc/ @biocom

alfetopito commented 3 years ago

I vote for onchain cancellation only for onchain orders.

I feel like it would confuse more than help most users to have 2 ways of cancelling EOAs.

We can iterate and add an "advanced" option later on if there's demand, but I wouldn't want to add it now.

anxolin commented 3 years ago

I vote for onchain cancellation only for onchain orders. I feel like it would confuse more than help most users to have 2 ways of cancelling EOAs. We can iterate and add an "advanced" option later on if there's demand, but I wouldn't want to add it now.

I'm also prefering that. Noone has expressed any concern with the other kind of cancelations. However, i left it open to hear if someone has more opinions.

fairlighteth commented 3 years ago

Here's my understanding:

Conclusion:

stale[bot] commented 2 years ago

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.