-
Notifications
You must be signed in to change notification settings - Fork 3
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
Enhance Dex functionality with new components and improved user experience #563
Closed
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
…st and selection handling
…h dynamic network details
…tokens dialog components
… selection in swapping page
…d ITokensDialogData types
…or token swapping functionality
…for adding liquidity
…re and service integration
…dd TODO for getPool implementation
… DexService methods
…DexService for pool retrieval
…val and dynamic contract address
…orm with conditional buttons
… enhance error handling in DexService
…erves from the pool
…nalities in the Dex service and store
…hance quote handling
…dress and streamline payload structure
…re for token swaps
…r token availability
…es with removal functionality
…reation state management
…and maintainability
…n approval process
…dity and swapping page functionality
…ects for loading allowance after approval
…transaction confirmation feedback
…transaction hash handling
… state management and form handling
…zation in liquidity and swapping pages
…liquidity and swapping pages
… path initialization in swapping page
…c in swapping page
…ton visibility logic
…ces for components
…ty and swapping pages
…into feature/258-dex
.../use-cases-page/pages/dex/containers/dex-liquidity-page/dex-liquidity-page.component.spec.ts
Dismissed
Show dismissed
Hide dismissed
.../use-cases-page/pages/dex/containers/dex-liquidity-page/dex-liquidity-page.component.spec.ts
Dismissed
Show dismissed
Hide dismissed
.../use-cases-page/pages/dex/containers/dex-liquidity-page/dex-liquidity-page.component.spec.ts
Dismissed
Show dismissed
Hide dismissed
...ge/use-cases-page/pages/dex/containers/dex-swapping-page/dex-swapping-page.component.spec.ts
Dismissed
Show dismissed
Hide dismissed
...ge/use-cases-page/pages/dex/containers/dex-swapping-page/dex-swapping-page.component.spec.ts
Dismissed
Show dismissed
Hide dismissed
UI/src/app/shared/components/modal/network-dialog/network-dialog.component.spec.ts
Dismissed
Show dismissed
Hide dismissed
UI/src/app/shared/components/modal/network-dialog/network-dialog.component.spec.ts
Dismissed
Show dismissed
Hide dismissed
UI/src/app/shared/components/modal/network-dialog/network-dialog.component.spec.ts
Dismissed
Show dismissed
Hide dismissed
props<{ payload: ILiquidityPayload }>() | ||
); | ||
export const removeLiquidityActionSuccess = createAction( | ||
'[Swap] Remove Liquidity Actions Success', // TODO: Add type here |
Check notice
Code scanning / devskim
A "TODO" or similar was left in source code, possibly indicating incomplete functionality Note
Suspicious comment
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.
Introduce a home page for Dex, enhance swapping and liquidity functionalities, and improve overall layout and styling. Implement dynamic network and token selection dialogs, along with transaction management features. Refactor code for better maintainability and integrate new state management for swaps and transactions.