feat: transaction store and toasts #2382
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
feat: replace useStoredEthereumTransaction with useEthTransactionManager feat: add event bus subsciption to vega transaction store feat: handle order cancellation feat: rename Deposit, Order and Withdraw status field to be unique Revert "feat: rename Deposit, Order and Withdraw status field to be unique" This reverts commit f0b314d. feat: split transaction update subscription feat: handle order and deposit transaction feat: handle withdrawal creation through transaction store feat: handle withdraw approval feat: handle panding withdrawls, add createdAt feat: handle transaction toast/dialog dismissal feat: add use vega transaction store tests feat: add use vega transaction store tests feat: add use vega transaction menager tests feat: add use vega transaction menager tests feat: add use vega transaction updater tests feat: improve use vega transaction updater tests feat: add use eth transaction store feat: add use eth withdraw approvals store feat: add use eth transaction updater tests fixed tests
feat: toasts feat: toasts
✅ Deploy Preview for liquidity-provision-dashboard ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
✅ Deploy Preview for vega-ui-toolkit ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
✅ Deploy Preview for vega-tools ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
✅ Deploy Preview for vega-trading-app ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
✅ Deploy Preview for market-depth ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
|
Test summaryRun details
View run in Cypress Dashboard ➡️ This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard |
MadalinaRaicu
approved these changes
Dec 9, 2022
✅ Deploy Preview for console-mainnet-mirror ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
bglownia
force-pushed
the
feat/transaction-store-and-toasts
branch
from
3339ce9
to
6ed7cee
Compare
mattrussell36
requested changes
Dec 14, 2022
|
Just noticed this z-index issue with the order book header
|
mattrussell36
approved these changes
Dec 20, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Related issues 🔗
Closes #1679
Demo 📺
Screen.Recording.2022-12-09.at.16.16.10.mov