- Issue created by @spiderman
- 🇨🇦Canada spiderman Halifax, NS
I noticed some bits were missing from the patch to make this work against the latest Workflow code. Attached is a new patch with interdiff showing added HookHandler bits etc.
- Status changed to Needs review
9 days ago 3:25pm 26 June 2025 - 🇳🇱Netherlands johnv
Attached patch is updated for ECA v2.1.8 and Workflow v2.1.
It also contains resolved incompatibilities between ECA v1 and ECA v2.
I did not truly test a 'model'.I took the liberty to:
- not make dependent on D11, but D11.2, since that is my dev environment
- not make dependent on Workflow version 1.8 (which is obsolete), 1.9 (which does not exist), 2.0 (which has had a short lifetime), but 2.1 (of which 2.1.2 is current version)It contains also 2 @todo's, where my IDE tells me there is a problem, I cannot fix myself.
- 🇳🇱Netherlands johnv
Please find attached a new version, after testing an Event + Action.
I did not find a way to test a Condition, though. - 🇳🇱Netherlands johnv
Please find attached a new version, after also having tested the 'WorkflowFieldValue' condition.
Some remarks:
- The 'WorkflowHasField' condition is not tested yet.
- The buildConfigurationForm from the Event is removed. IMO events have no configuration
- The buildConfigurationForm from the Condition - I do not understand how the UI gets its values. The code has no default values, but the UI has.
- There are some @todo's.Sorry for not using MR.