bptlab / chor-js-demo

A demo showcasing the chor-js BPMN 2.0 choreography diagram modeler.
23 stars 10 forks source link

Extend validator rules #41

Open jan-ladleif opened 4 years ago

jan-ladleif commented 4 years ago

We should add additional validation rules, now that we can specify ItemDefinitions and such. What I am thinking of from the top of my head:

Proposed Methodology

  1. Go through the standard systematically and collect all constraints.
  2. Consolidate these constraints, look for overlaps, etc.
  3. Discuss whether we can check these constraints.
  4. Implement those which we do not have yet.
jan-ladleif commented 4 years ago

Timer event validation was added in #46.

However, the standard does not at all make sense: image would allow diagrams such as this: image But obviously, C does not know whether the upper path has been taken, and just starting c at the first of January would not always be compliant.

jan-ladleif commented 3 years ago

The standard is too vague, PR #47 is closed.