-
Notifications
You must be signed in to change notification settings - Fork 984
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: implement proper error handling for swaps #21261
base: develop
Are you sure you want to change the base?
Conversation
Jenkins BuildsClick to see older builds (41)
|
fb68f41
to
232576a
Compare
Hi @briansztamfater, should this PR be tested now, or is it better to wait until the status changes from 'draft' to 'open'? |
Hey @VolodLytvynenko, this is almost ready, but I think it is better to first verify #21179 is correctly tested, because status-go latest changes introduced some bugs in the mobile client and would make difficult to test this PR. Once we merge #21179 I will rebase this one, solve the conflicts and send it to review. |
645c3b1
to
b22b49a
Compare
b22b49a
to
90488f5
Compare
Signed-off-by: Brian Sztamfater <[email protected]>
8c75a74
to
50343db
Compare
fixes #20342
fixes #20271
Summary
This PR implements error handling for swap proposal errors.
Platforms
Areas that maybe impacted
Functional
Steps to test
status: ready