Email for order pending/fill/target/stop

[NinjaTrader_JoshP]

This is a basic C# concept. The sample is correct because it is only doing one line of code for the if-statement. If you want to do multiple lines of code you need to use brackets.

if (...)
{
    _____;
    _____;
}

[m3power222]

This is a basic C# concept. The sample is correct because it is only doing one line of code for the if-statement. If you want to do multiple lines of code you need to use brackets.

if (...)
{
    _____;
    _____;
}

Over and over again, Josh thx.

[m3power222]

You would need to work with IOrder objects for this, here's a sample demonstrating this - http://www.ninjatrader-support2.com/...ead.php?t=7499

I don't see anything in there that looks for Target1 or Stop1...

[NinjaTrader_JoshP]

To clarify, OnOrderUpdate() is not for ATM managed orders.

[m3power222]

To clarify, OnOrderUpdate() is not for ATM managed orders.

Then it appears to be GetAtmStrategyStopTargetOrderStatus().

[NinjaTrader_JoshP]

Correct. That will be the method.

[m3power222]

Correct. That will be the method.

Just trying to print to the output window but it doesn't work:

// Print out some information about the target to the output window
string[,] orders = GetAtmStrategyStopTargetOrderStatus("TARGET1", "idValue");

// Check length to ensure that returned array holds order information
if (orders.Length > 0)
{
for (int i = 0; i < orders.GetLength(0) - 1; i++)
{
Print("Average fill price is " + orders[i, 0].ToString());
Print("Filled amount is " + orders[i, 1].ToString());
Print("Current state is " + orders[i, 2].ToString());
}
}

[NinjaTrader_JoshP]

idValue is likely your variable and not the actual string you are trying to pass in. Same issue as you had before.

[m3power222]

idValue is likely your variable and not the actual string you are trying to pass in. Same issue as you had before.

I do have this:

private string idValue = string.Empty;

I'm not sure why it isn't working.

[NinjaTrader_JoshP]

Same reason as the error before. You are using quotes when you should just be calling the variable directly.

[m3power222]

Same reason as the error before. You are using quotes when you should just be calling the variable directly.

Removed the quotes and still does not work.

[m3power222]

Changed it similar structure to the GetAtmStrategyEntryOrderStatus in the SampleAtmStrat:

// Check for a pending entry order
if (atmStrategyId.Length > 0)
{
string[,] orders = GetAtmStrategyStopTargetOrderStatus("Target1", atmStrategyId);

// Check length to ensure that returned array holds order information
if (orders.Length > 0)
{
for (int i = 0; i < orders.GetLength(0) - 1; i++)
{
Print("Average fill price is " + orders[i, 0].ToString());
Print("Filled amount is " + orders[i, 1].ToString());
Print("Current state is " + orders[i, 2].ToString());
SendMail("[email protected]", "[email protected]", "status", orders[i, 2].ToString());
}
}
}

In the log highlighted in yellow it says:

GetAtmStrategyStopTargetOrderStatus() method error: OrderName 'Target1' does not exist

I assumed since my ATM Strategy that is called has 2 targets it would automatically have Target1/Target2/Stop1/Stop2.

[NinjaTrader_Bertrand]

Please try

string[,] orders = GetAtmStrategyStopTargetOrderStatus("TARGET1", atmStrategyId);

[m3power222]

Please try

string[,] orders = GetAtmStrategyStopTargetOrderStatus("TARGET1", atmStrategyId);

That is what is already in place, see above post.

[NinjaTrader_JoshP]

Please try with all caps. "TARGET1"