Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error Messages werden nicht angezeigt #942

Closed
1 task
darenegade opened this issue Nov 10, 2023 · 5 comments · Fixed by #1036 or #1249
Closed
1 task

Error Messages werden nicht angezeigt #942

darenegade opened this issue Nov 10, 2023 · 5 comments · Fixed by #1036 or #1249
Assignees
Labels
bug Something isn't working digiwf-apps internal

Comments

@darenegade
Copy link
Member

darenegade commented Nov 10, 2023

Describe the bug
In der CoCreation angegebene Error-Messages werden im Formular nicht angezeigt

To Reproduce
Steps to reproduce the behavior: Sie internal Ticket

Acceptance Criteria

  • Error-Messages werden angezeigt

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

@darenegade darenegade added bug Something isn't working digiwf-apps internal labels Nov 10, 2023
@lmoesle lmoesle self-assigned this Nov 21, 2023
@lmoesle
Copy link
Contributor

lmoesle commented Nov 21, 2023

Form: bugfix-942-error-msg.json

@lmoesle
Copy link
Contributor

lmoesle commented Nov 21, 2023

Die Formbuilder Settings waren falsch gesetzt. Dadurch wurde die message Konfiguration für das Textfeld an der falschen Stelle eingefügt.

Bei schon bestehenden Formularen muss händisch das Json Schema angepasst werden, indem die Message Konfiguration gesetzt werden.
Sobald wir eine neue Version des Formbuilders veröffentlicht haben, wird die Konfiguration automatisch richtig gesetzt.

Richtig

"aktenzeichen": {
  "fieldType": "text",
  "title": "WM-Wohnungsnummer",
  "type": "string",
  "key": "aktenzeichen",
  "description": "Format S12-123456",
  "x-options": {
    "fieldColProps": {
      "cols": 12,
      "sm": 12
    },
    "messages": {
      "pattern": "Das Muster S12-123456 wird nicht eingehalten",
      "minLength": "Die Nummer ist kürzer als {minLength} Zeichen, bitte überprüfen Sie diese."
    }
  },
  "x-props": {
    "outlined": true,
    "dense": true
  },
  "x-rules": [],
  "pattern": "S\\d{2}-\\d+",
  "minLength": 4
}
}

Die message Konfiguration sollte nicht in den fieldColProps definiert werden.

Falsch

"properties": {
  "aktenzeichen": {
    "fieldType": "text",
    "title": "WM-Wohnungsnummer",
    "type": "string",
    "key": "aktenzeichen",
    "description": "Format S12-123456",
    "x-options": {
      "fieldColProps": {
        "cols": 12,
        "sm": 12,
        "messages": {
          "pattern": "Das Muster S12-123456 wird nicht eingehalten",
          "minLength": "Die Nummer ist kürzer als {minLength} Zeichen, bitte überprüfen Sie diese."
        }
      }
    },
    "x-props": {
      "outlined": true,
      "dense": true
    },
    "x-rules": [],
    "pattern": "S\\d{2}-\\d+",
    "minLength": 4
  }
}

@darenegade
Copy link
Member Author

Dass das erst jetzt auffällt 🙈🤣

@simonhir
Copy link
Member

simonhir commented Jan 24, 2024

Funktioniert aktuell noch nicht für komplexe Objekte. Mindestens array, arrayObject und wahrscheinlich auch multi-user-input

@simonhir simonhir reopened this Jan 24, 2024
@lmoesle lmoesle self-assigned this Jan 25, 2024
@lmoesle
Copy link
Contributor

lmoesle commented Jan 25, 2024

Funktioniert aktuell noch nicht für komplexe Objekte. Mindestens array, arrayObject und wahrscheinlich auch multi-user-input

In dem Fall heißen die Configs minItems und maxItems. Ich passe das im Formbuilder auch noch an

lmoesle added a commit that referenced this issue Feb 2, 2024
* bugfix(#942): fix form builder settings for array types

* bugfix(#942): fix form builder settings for array types

* bugfix: add message config for arrayObject type fields

* bugfix(#942): ajdust form builder settings for all field types
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working digiwf-apps internal
Projects
None yet
3 participants