Netz39_Vorstand/entities_validation_svc
3
0
Fork 0
Code Issues Pull requests 1 Releases 1 Activity

WIP: validators #3

Closed
dkdent wants to merge 6 commits from validators into master
pull from: validators
merge into: Netz39_Vorstand:master
Conversation 4 Commits 6 Files changed 3 +114
dkdent commented 2020-11-07 19:27:37 +01:00
Collaborator
Copy link
No description provided.
tux was assigned by dkdent 2020-11-07 19:27:37 +01:00
tux changed title from validators to WIP: validators 2020-11-07 20:24:57 +01:00
dkdent changed title from WIP: validators to validators 2020-11-11 18:01:50 +01:00
tux commented 2020-11-29 14:28:28 +01:00
Owner
Copy link

Dieser PR ist für mich eigentlich okay, ich denke nur noch über eine Sache nach:
Wenn die Validatoren sowieso in einem Unterverzeichnis sind, warum dann nicht auch in getrennten Dateien?

Ist es evtl sinnvoll, gleich ein Python-Modul daraus zu machen?

Dieser PR ist für mich eigentlich okay, ich denke nur noch über eine Sache nach: Wenn die Validatoren sowieso in einem Unterverzeichnis sind, warum dann nicht auch in getrennten Dateien? Ist es evtl sinnvoll, gleich ein Python-Modul daraus zu machen?
dkdent commented 2020-11-29 14:35:22 +01:00
Author
Collaborator
Copy link

Ja, da hab ich auch schon drüber nach gedacht und bin noch nicht zu einem befriedigenden Ergebnis gekommen. An sich würde ich auch gern eine Möglichkeit vorsehen, um der Antwort ein finding mitzugeben, damit der Nutzer nicht nur mit einem roten Licht da steht...

Ja, da hab ich auch schon drüber nach gedacht und bin noch nicht zu einem befriedigenden Ergebnis gekommen. An sich würde ich auch gern eine Möglichkeit vorsehen, um der Antwort ein finding mitzugeben, damit der Nutzer nicht nur mit einem roten Licht da steht...
tux commented 2020-11-29 21:37:14 +01:00
Owner
Copy link

Ja, da hab ich auch schon drüber nach gedacht und bin noch nicht zu einem befriedigenden Ergebnis gekommen. An sich würde ich auch gern eine Möglichkeit vorsehen, um der Antwort ein finding mitzugeben, damit der Nutzer nicht nur mit einem roten Licht da steht...

Ja, dazu wollte ich die API auch noch einmal anpassen. Wir brauchen eine schönere Möglichkeit, um einerseits Fehler und Warnungen zu unterscheiden und andererseits strukturierter zu sagen, worauf sich eine Meldung bezieht. Derzeit können wir pro Feld maximal einen Fehler zurückgeben.

> Ja, da hab ich auch schon drüber nach gedacht und bin noch nicht zu einem befriedigenden Ergebnis gekommen. An sich würde ich auch gern eine Möglichkeit vorsehen, um der Antwort ein finding mitzugeben, damit der Nutzer nicht nur mit einem roten Licht da steht... Ja, dazu wollte ich die API auch noch einmal anpassen. Wir brauchen eine schönere Möglichkeit, um einerseits Fehler und Warnungen zu unterscheiden und andererseits strukturierter zu sagen, worauf sich eine Meldung bezieht. Derzeit können wir pro Feld maximal einen Fehler zurückgeben.
dkdent commented 2020-12-03 14:50:05 +01:00
Author
Collaborator
Copy link

Ich würde das hier zu machen, da die Funktionen mit der Signatur nicht zu den Validation-Frameworks passen werden.

/close

Ich würde das hier zu machen, da die Funktionen mit der Signatur nicht zu den Validation-Frameworks passen werden. /close
dkdent changed title from validators to WIP: validators 2020-12-03 14:50:48 +01:00
dkdent closed this pull request 2020-12-04 19:09:20 +01:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
tux
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Netz39_Vorstand/entities_validation_svc#3
No description provided.
Delete branch "validators"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?