Skip to content

Conversation

@GuillaumeDSM
Copy link
Member

No description provided.

@GuillaumeDSM GuillaumeDSM requested a review from Herklos November 7, 2025 21:41
@GuillaumeDSM GuillaumeDSM self-assigned this Nov 7, 2025
@GuillaumeDSM GuillaumeDSM force-pushed the cancel_policy branch 4 times, most recently from a8eb2de to bf22e41 Compare November 11, 2025 21:51
@GuillaumeDSM GuillaumeDSM changed the title WIP add Cancel policies add Cancel policies Nov 12, 2025
@GuillaumeDSM GuillaumeDSM force-pushed the cancel_policy branch 2 times, most recently from 59963d7 to 460d583 Compare November 12, 2025 10:23
Copy link
Member

@Herklos Herklos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice feature 👍


policy = chained_order_filling_price_order_cancel_policy_import.ChainedOrderFillingPriceOrderCancelPolicy()

# current_price (90) < filling_price (100) and trigger_above=False, should cancel
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@GuillaumeDSM GuillaumeDSM merged commit c0897d9 into master Nov 12, 2025
5 checks passed
@GuillaumeDSM GuillaumeDSM deleted the cancel_policy branch November 12, 2025 21:35
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.

3 participants