Feat/validation errors #126
Conversation
tumidi
force-pushed
the
feat/validation-errors
branch
from
541e7c0
to
2716daa
Compare
| if (!element_el) { | ||
| throw new Error(`Could not find form field '${field}'!`); | ||
| } | ||
|
|
||
| const errors_el = get_errors_el(element_el); | ||
| if (!errors_el) { | ||
| throw new Error(`Could not find errors element for field '${field}'!`); | ||
| } |
There was a problem hiding this comment.
Ich denke hier wären wir mit console.error und continue besser bedient.
There was a problem hiding this comment.
Ich benutze hier mit voller Absicht throw new Error. Es handelt sich um critical error conditions. Sollte so ein Fall eintreten, handelt es sich um einen Bug, der im besten Fall reportet wird. Bei Verwendung von console.error/continue würde das Script ja einfach weiterlaufen und welcher User überwacht schon permanent die dev-Konsole des Browsers und nimmt Fehler dort ernst? Bugs werden dann reportet wenn Dinge offensichtlich und hart failen.
Für eine bessere UX lassen die Exceptions sich außerdem fangen und in einem übersichtlichen User dialog anzeigen, mit der Bitte den Bug zu reporten, unter Anzeige eines Stack trace, Link zum Bug-Tracker, etc. (was nicht Scope dieses PRs ist).
A text input with no content should be considered unfilled.
…ing_question_from_invalid_options`
tumidi
force-pushed
the
feat/validation-errors
branch
from
a00338a
to
29912da
Compare
Ruff-Fehler erwartet (Dependency management & tooling #123)