Update trigger price validation

[ghost]

Fixes #5979

[alkum]

ACK.

Tested with a -5% market deviation and cutoff price of 41 000 USD.

Before this PR:

After PR:

[ripcurlx]

I'd rather use the MarketPrice object exposing the currencyCode property instead of adding another parameter. WDYT?

[ghost]

Good idea - fixed

[ripcurlx]

@xyzmaker123 Did you also check the trigger price actual execution code? I'd surprised, if only changing the UI validation will do the work as expected.

[ghost]

Did you also check the trigger price actual execution code? I'd surprised, if only changing the UI validation will do the work as expected.

@ripcurlx I roughly reviewed it. I think changing validation criteria is enough - we basically add more flexibility to user to set trigger price possible values, but execution logic doesn't change.

[ripcurlx]

True - just looked at the code myself. There doesn't seem to be any special handling that matches the one of the validation before.

[ripcurlx]

ACK - tested UI validation on Regtest. Behavior for positiv difference to market does not change only for negative difference (against your advantage).