search

it-at-m / digiwf-core

central workflow automation and integration platform based on the free process framework Camunda.
MIT License
19 stars 7 forks source link

Multi-User Input: Required broken #442

Closed darenegade closed 1 year ago

darenegade commented 1 year ago

Describe the bug Multi-User Input kann nicht als Pflichtfeld definiert werden

To Reproduce Steps to reproduce the behavior:

  1. Multi-User Input in Form und versuchen als Pflichtfeld zu definieren
  2. Multi-User ist kein Pflichtfeld

Expected behavior Pflichtfeld kann bei MultiUser Input gesetzt werden und wird im Formular validiert

Additional context https://git.muenchen.de/digitalisierung/digiwf-support/-/issues/284

simonhir commented 1 year ago

Glaube nicht, dass das ein wirklicher Bug ist, da mit required nur auf leere Strings getestet wird und hier der Wert ja wenn [] ist. Siehe https://github.com/it-at-m/vuetify-jsonschema-form/blob/master/lib/utils/rules.js#L6

Dafür gibts ja dann eigentlich extra minItems und maxItems.

Mein Vorschlag wäre hier und bei #440 das required Feld weg zu machen (wenn vorhanden) damit bei beiden nur min und max Items gewählt werden kann.

darenegade commented 1 year ago

@simonhir Vorschlag wäre okay und können wir gerne machen.