• If this is your first visit, you will have to register before you can post. To view messages, please scroll below and select the forum that you would like to visits. Questions? Be sure to check out the Forum FAQ.

Announcement

Collapse
No announcement yet.

Partner 728x90

Collapse

Request to make IsTerminalState() supported

Collapse
X
 
  • Filter
  • Time
  • Show
Clear All
new posts

    Request to make IsTerminalState() supported

    I assume that Order.IsTerminalState was written to make code inside SubmitOrderManaged() more concise and readable. Well the same thing applies to user code, so please make it supported.

    Example supported code
    Code:
    if (entryOrder.OrderState==OrderState.Cancelled || entryOrder.OrderState==OrderState.Filled || entryOrder.OrderState==OrderState.Rejected || entryOrder.OrderState==OrderState.Unknown)
    can be replaced with the much clearer (but currently unsupported)
    Code:
    if(Order.IsTerminalState(entryOrder.OrderState))

    #2
    Hello,

    Thank you for the post.

    I will forward this to development as a feature request.

    I look forward to being of further assistance.
    JesseNinjaTrader Customer Service

    Comment


      #3
      I see IsTerminalState has been added to the documentation. Thanks to whoever did that.

      Comment

      Latest Posts

      Collapse

      Topics Statistics Last Post
      Started by Benjaminf, Today, 02:21 PM
      0 responses
      4 views
      0 likes
      Last Post Benjaminf  
      Started by chrismcdermut, Today, 01:46 PM
      1 response
      6 views
      0 likes
      Last Post NinjaTrader_Kate  
      Started by dpolyakov, Today, 01:13 PM
      3 responses
      14 views
      0 likes
      Last Post dpolyakov  
      Started by tjendra, Today, 12:15 PM
      3 responses
      9 views
      0 likes
      Last Post tjendra
      by tjendra
       
      Started by lunardiplomacy, Today, 11:46 AM
      4 responses
      17 views
      0 likes
      Last Post lunardiplomacy  
      Working...
      X