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

Adds Pre and Post Tradier Order Duration #17

Open
4 tasks done
AlexCatarino opened this issue Apr 1, 2022 · 3 comments
Open
4 tasks done

Adds Pre and Post Tradier Order Duration #17

AlexCatarino opened this issue Apr 1, 2022 · 3 comments

Comments

@AlexCatarino
Copy link
Member

Expected Behavior

Be able to place limit order outside market hours with Tradier:
https://blog.tradier.com/blog/2018/05/trading-in-the-pre-and-post-market-sessions-tradier-brokerage.html

Actual Behavior

It's not possible.
Additionally, if there is one of these orders in the account, Lean throws an exception:

Error converting value "pre" to type 'QuantConnect.Brokerages.Tradier.TradierOrderDuration'

Potential Solution

N/A

Reproducing the Problem

Please a limit order with the pre duration:
pre
and deploy LEAN before it is filled.

Checklist

  • I have completely filled out this template
  • I have confirmed that this issue exists on the current master branch
  • I have confirmed that this is not a duplicate issue by searching issues
  • I have provided detailed steps to reproduce the issue
@AlexCatarino AlexCatarino added the bug Something isn't working label Apr 1, 2022
@bryan78305
Copy link

How far back does the synchronization go? The error is pulling trade data back to "create_date":"2022-03-31T20:00:26.865Z"

The error is not reliant on the trade being unfilled when LEAN is deployed. The Pre-market trade was placed and filled prior to starting LEAN.

@Martin-Molinero
Copy link
Member

Hey @bryan78305! Sorry about that, issue should be solved now by PR QuantConnect/Lean#6281. Leaving this issue open as feature to add support for Pre and Post order duration for tradier.

@jaredbroad jaredbroad transferred this issue from QuantConnect/Lean Feb 16, 2023
@bryan78305
Copy link

bryan78305 commented Feb 16, 2023 via email

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

No branches or pull requests

3 participants