Skip to content
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

New swapper flow - wire-up status message #8548

Closed
gomesalexandre opened this issue Jan 13, 2025 · 2 comments · Fixed by #8649
Closed

New swapper flow - wire-up status message #8548

gomesalexandre opened this issue Jan 13, 2025 · 2 comments · Fixed by #8649
Assignees

Comments

@gomesalexandre
Copy link
Contributor

Overview

Currently with the new swapper flow flag, the message property of checkTradeStatus isn't wired up anymore.
This is mostly visible with Li.Fi bridges, where we use checkTradeStatus as a long-poll of sorts to surface the current swap status.

References and additional details

message: statusResponse.substatusMessage,

Acceptance Criteria

  • The message property is wired-up again
  • Li.Fi long-polled status updates are happy again
  • THOR DCA and Chainflip streaming progress is happy

Need By Date

No response

Screenshots/Mockups

https://jam.dev/c/31b2a8a0-f5ac-42b2-b3e4-e0c1d99cd268

Estimated effort

No response

@0xean
Copy link
Contributor

0xean commented Jan 13, 2025

@shapeshift/product - please review - we are moving forward with showing the status messages below current state, but welcome additional changes or a new ticket to revise further.

@0xean
Copy link
Contributor

0xean commented Jan 13, 2025

Screenshot 2025-01-13 at 6 49 50 PM

Substeps from the current state:

"Awaiting inbound tx"
"Awating ... "

? 'Outbound transaction transmitted, waiting on destination chain...'

@0xean 0xean added the needs product Requires product input before bounty label Jan 13, 2025
@twblack88 twblack88 removed the needs product Requires product input before bounty label Jan 20, 2025
@gomesalexandre gomesalexandre self-assigned this Jan 22, 2025
@gomesalexandre gomesalexandre moved this from Up next / groomed to In review in ShapeShift Dashboard Jan 22, 2025
@github-project-automation github-project-automation bot moved this from In review to Done in ShapeShift Dashboard Jan 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants