Corey McKrill
Corey McKrill
I _think_ we're on the same page here, but just to reiterate, if you're building an order in the mobile app, and you add a coupon (regardless of whether it...
I updated the testing instructions and the changelog entry. Going to close and reopen this to restart the stuck CI checks.
Thanks again for the PR, @kkmuffme !
Hey @79mplus-admin thanks for the PR! I'm actually wondering if there's ever a time when you're setting an order status via the API and you _wouldn't_ want it to be...
@79mplus-admin Right. What I'm trying to get at is, if we imagine that this "manual" parameter already existed, would you ever _not_ use it and set it to true?
@79mplus-admin I thought about this some more, and I think you're right, since "system" is already the default, we shouldn't change that, so having the `manual_update` parameter is probably the...
I'm going to close and reopen this issue to get the CI checks unstuck.
Noting that I updated the testing instructions to be more detailed.
Thanks again for the PR, @79mplus-admin !
@JacklynBiggin checking in about this docs update. It would be great to get this merged ASAP now that 8.6 is out. Is there anything you're waiting on me for?