juice-interface icon indicating copy to clipboard operation
juice-interface copied to clipboard

Handle all transactor failures gracefully

Open benschac opened this issue 3 years ago • 2 comments

Description

We get a bunch of issues that are related to a transaction request failing, getting stuck in a loading state, not showing, and errors that don't notify users.

Proposal

I wanted to go through src/hooks/v1|v2/transactor and add try catch blocks / error handling so users will get error messages with detailed failure states + don't get stuck in loading states when any of these API requests fail.

Screenshots

If applicable, add screenshots to help explain your problem.

benschac avatar Aug 12 '22 15:08 benschac

https://github.com/jbx-protocol/juice-interface/pull/1755#discussion_r955054199 on #1755 more context. tl;dr moving to juice-sdk and we can just do these check explicitly

benschac avatar Aug 25 '22 14:08 benschac

cc: @NakedFool -- this is the ticket I was talking about in https://github.com/jbx-protocol/juice-interface/pull/1791

benschac avatar Aug 25 '22 14:08 benschac