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

Feature/twap UI tradingpost #37

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

oxbyte-monterrosa
Copy link

@oxbyte-monterrosa oxbyte-monterrosa commented Jul 26, 2024

Purpose of this PR

This PR aims to introduce a new package for @orbs-network/twap-ui, specifically designed for a new DEX "Trading Post"
@orbs-network/twap-ui-tradingpost

Context

PancakeSwap is used as the base platform for leverage, but the intention is to completely change the UI to suit the needs of Trading Post (TWAP and limit orders).

@oxbyte-monterrosa oxbyte-monterrosa force-pushed the feature/twap-ui-tradingpost0.10.14 branch from b2645e6 to a9e402d Compare August 5, 2024 18:00
- Introduced a `dex` prop to the `OrdersList` component to support different decentralized exchanges (DEXs).
- Implemented custom behavior for the `TradingPost` DEX, rendering a specific `OrdersList` with customized props.
- Updated the `SelectedOrders` component to utilize the new `dex` prop, allowing dynamic rendering based on the selected DEX.
- Refactored and added JSDoc documentation for better clarity and maintainability.
- Removed obsolete elements and ensured a cleaner, more efficient design.
- Updated and reorganized UI components to align with the new package structure.
- Deleted the custom module declaration for SVG imports.
- Going forward, SVG files will be wrapped in dedicated React components instead of being imported directly.
@oxbyte-monterrosa oxbyte-monterrosa marked this pull request as ready for review August 21, 2024 01:37
@oxbyte-monterrosa oxbyte-monterrosa changed the title Feature/twap UI tradingpost0.10.14 Feature/twap UI tradingpost Aug 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant