Corey McKrill

Results 93 comments of Corey McKrill
trafficstars

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.

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?