forbole / forbole-x

A multichain asset management platform.
https://www.forbole.com/products
Apache License 2.0
21 stars 15 forks source link

Allow users to set gas manually #591

Closed leobragaz closed 2 years ago

leobragaz commented 2 years ago

While making a tx, users can't set the gas manually. This can lead to problems like this one: rejected

I suggest to add a button "Set gas" directly on the extension to allow users to set it manually. I imagine something like this: gas

kwunyeung commented 2 years ago

@bragaz do you know which tx the user is trying to broadcast. We should have applied gas adjustment to the gas estimate before signing the tx. I'm wondering how much gas is needed for this tx.

Instead of setting a gas value directly, I would suggest showing the gas estimate and allow users to change the gas adjustment with a slider.

leobragaz commented 2 years ago

@kwunyeung most of the users are facing it while trying to claim rewards. So a withdraw-rewards / withdraw_all_rewards transaction. I've also registered some cases of this problem with the unbound transaction.

kanniec826 commented 2 years ago
nicoletting commented 2 years ago

As discussed last time, I found 3 solutions for the UI: image.png I think (C) is the most suitable and educative one, for serval reasons:

  1. present concept that complex transactions require more gas and the wanted gas range
  2. provide 3 buttons for user's daily practice but in the different definition -> "simple, medium, complex"
  3. adjustable slider & enable input amount and adjust amount with [+], [-] button.

to review the prototype please visit here @kwunyeung @kanniec826

kanniec826 commented 2 years ago

@nicoletting Personally I prefer A

kanniec826 commented 2 years ago

Please add your planning poker estimate with ZenHub @nicoletting

kanniec826 commented 2 years ago

Please add your planning poker estimate with ZenHub @calvinkei

calvinkei commented 2 years ago

actually do we still need to do this? the error is caused by gas fee rounding instead of insufficient gas. 1.5x should be more than enough and no user is reporting this issue again since the fix

kwunyeung commented 2 years ago

@calvinkei @nicoletting maybe this should be treated as a separate issue.

My concern is in case 1.5x is not enough in the gas estimate, there is still option to override the gas amount. So for this case, I prefer a more simple way to set the gas but not having 1.5, 1.6, 1.7 for choosing. Having these options are meaningless.

nicoletting commented 2 years ago

I've modified the UI and it is now ready: view prototype here

please change UI of delegate/ redeledate/ undelegate/ withdraw rewards/ send🙇🏻‍♀️ icon_edit_light icon_edit_dark