Define required constraint for BPMN based modellers

Nice, thanks for providing this. I just tested this locally, and works.

I'm not sure whether the current UI is what's expected to be good UX. That's a subjective thing I guess, for me this looks unfamiliar. Because the field is just marked red with a text "Must not be empty.", which may look like the user did something wrong (but didn't).

Yes, the UX is debatable. But it's not an ECA issue. ECA just provides the information which fields are required. What the modeller is doing with that information is out of scope. In case of the bpmn.io modeller, this is addressed in 📌 Required form field should be marked as mandatory Postponed: needs info .

Looks like this is then RTBC and I wonder if it needs to be back ported?

Yes, could be back ported if you want. As this works, setting RTBC. I'm a bit concerned that people will find this confusing, but we could see how the feedback is (if any).

Merged and back ported to 1.1.x only.

Automatically closed - issue fixed for 2 weeks with no activity.