feat(Notification): better validation #400
Conversation
timreibe
changed the title
|
kann das rein? @marcorombach |
Falls das oben beschriebene Problem fürs erste akzeptabel ist, kanns rein. Falls mir dazu eine Lösung einfällt, reiche ich die nach. |
|
@iamnotturner Nein, das darf auf keinen Fall rein. Hier werden die |
Alles klar, ich baue das so um, dass alles interaktive in update_kontaktdaten_interactive stattfindet. |
tools/kontaktdaten.py
Outdated
| from tools.utils import telegram_validation, pushover_validation | ||
| from tools.exceptions import ValidationError, MissingValuesError | ||
| from tools import Modus |
There was a problem hiding this comment.
Am besten alphabetisch sortieren
|
Code sieht gut aus. Es sind aber teilweise unnötige Diffs im PR. |
marcorombach
force-pushed
the
notification_validation
branch
from
ddd9a2f to
229ba51
Compare
|
Habe mal einen squash-commit gepusht |
marcorombach
force-pushed
the
notification_validation
branch
from
df50ab3 to
7f2331f
Compare
|
@marcorombach Schön umgesetzt, besonders mit der Validierung der Daten. Dadurch kann man schon im Vorhinein Fehler bei der Kommunikation über Pushover/Telegram verhindern 👍 |
marcorombach
force-pushed
the
notification_validation
branch
from
7f2331f to
9d2f1dd
Compare
Danke :) Habe gerade noch ein paar kosmetische Änderungen hinzugefügt |
|
Dann ist das fertig oder @marcorombach |
Ja, ist fertig @sinun98 |
Bekanntes Problem: bei fehlerhafter Eingabe wird nicht die gesamte pushover oder telegram Eingabe von vorne gestartet. Die Struktur von input_kontaktdaten_key macht das schwierig.