David Cook
David Cook
Still a good question, maybe run it for 20 (will take a while), then review the commits. If there's some big ones, maybe break them up into separate branches/PRs.
I think this could be bigger, because #9616 only includes provision for _one_ webhook, so it won't be as simple as creating one more event type. I'll estimate: 1. Add...
**Question: can a user add more than one of each type of webhook?** (eg add Order Cycle Closed twice, once to Zapier and once to N8N). I think yes, it...
> You can always keep the old class for compatibility and use the new code: Thanks, I didn't think through if there might be other things to consider. I have...
You're right, it will need to be enabled for the user (that is, the owner of the coordinator/distributor enterprise). As it is currently, you need to log in as that...
I think @mkllnk and I agreed on papercut (from memory this means under 1 day development time). I'm assuming 2hrs of testing doesn't change that. I just realised that this...
Requires fix, see the [dependency updates guide](https://github.com/openfoodfoundation/openfoodnetwork/wiki/Dependency-updates).
😞 Yup, all Reflexes failed, for both v5.0.2 and v5.0.3 commits (evident in [failed-tests-screenshots](https://github.com/dacook/openfoodnetwork/actions/runs/8167360156/artifacts/1301036303)) Probably due to the version "mismatch"
It's very strange. Currently on `master`, the Gemfiles say: ``` gem "cable_ready", "5.0.1" cable_ready (= 5.0.1) ``` And package.json / yarn.lock: ``` "cable_ready": "5.0.2", [email protected], cable_ready@^5.0.0: version "5.0.2" ``` Yet...
Yay, after upgrading stimulus_reflex, this now works!