-
Notifications
You must be signed in to change notification settings - Fork 216
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
Add Maker mode, and order form options for GoodTillCancelled
/ FillOrKill
#2424
Conversation
GoodTillCancelled
/ FillOrKill
GoodTillCancelled
/ FillOrKill
Closes https://github.com/KomodoPlatform/komodo-wallet-desktop/issues/956 (the rest) makertaker.mp4
To test:
cancelmaker.mp4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Functionally everything checks out. However there is a UI issue on the Exchange Confirmation dialogue:
As seen in the image above, the "Cancel" and "Confirm" buttons overflow off the modal.
This is likely not necessarily an introduced issue. Although this is not seen on the previous versions, I believe the addition of the new checkbox pushes the buttons to the point where it shows.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Testing was successful, everything is working and looks good now.
The following items were tested:
- Sell Mode [Good until cancelled] checkbox is checked. Order remains in the Order Book
- Sell Mode [Good until cancelled] checkbox is not checked. Order does not remain in the Order Book after failing to match.
- Buy Mode [Good until cancelled] checkbox is checked. Order remains in the Order Book
- Buy Mode [Good until cancelled] checkbox is not checked. Order does not remain in the Order Book after failing to match.
- Place multiple orders in maker mode that are too high to match [Cancel Existing Orders] checkbox is not checked. Orders remain in order book
- Place an order in maker mode after the previous step with [Cancel Existing Orders] checkbox checked. All previous orders of the same type are cancelled, leaving only the current created order
- General UI testing throughout the module
- Functional testing of module features
Closes https://github.com/KomodoPlatform/komodo-wallet-desktop/issues/956
This PR also includes some changes to the dex pro view layout. All trading operations on both Simple and Pro view should be thoroughly tested for regression.
gtc.mp4
Beyond the layout changes, this PR introduces a checkbox in the
Confirm Exchange Details
modalThis checkbox is on by default, which is effectively the same as before: When you create an order, if it does not match within 30 seconds, it becomes a maker order and stays on the orderbook.
If the checkbox is unticked, the order id place with the
FillOrKill
order_type parameter.This means that if an order is placed which is not matched with an existing order on the orderbook, it will automatically cancel after 30 seconds.
To Test (pro view):
To Test (general):
PART 2 BELOW