validatie-architectuur-fix #229

Open
anna wants to merge 12 commits from validatie-architectuur-fix into dev
Collaborator

Wat doet dit hoofdzakelijk:

  • bugs van frontend data laden gefixt
  • lege JSON foutmeldingen in de browser console gefixt door t backend bewust iets leegs te laten terugsturen
  • services niet steeds verschillende nietszeggende types als string laten returnen voor de endpoints
  • centraal type ServiceResult met extension voor services aangemaakt zodat ik niet op 100000000000000000000000000000000 plekken aparte result codes moet teruggeven aan de endpoints en ik heb nu ook gewoon ez foutberichten
  • centrale frontend foutmelding-afhandelaar toegevoegd zodat ik dat ook niet op 10000000000000000 plekken moet doen
  • kleine validatiebugs in de backend opgelost
  • de IStep.Validate()-functie doet nu alleen een laatste check of er geen lege verplichte velden meer zijn. De set-service functies doen dat alleen al... ik moet nog even kijken of ik deze functie niet gewoon kan verbannen. De naam is ook wel vaag "Validate" is hier eigenlijk niets meer dan "valideer of er nog lege velden zijn voordat we doorgaan naar de volgende stap". ... dat zou een volgende fix zijn in de architectuur
Wat doet dit hoofdzakelijk: - bugs van frontend data laden gefixt - lege JSON foutmeldingen in de browser console gefixt door t backend bewust iets leegs te laten terugsturen - services niet steeds verschillende nietszeggende types als string laten returnen voor de endpoints - centraal type ServiceResult met extension voor services aangemaakt zodat ik niet op 100000000000000000000000000000000 plekken aparte result codes moet teruggeven aan de endpoints en ik heb nu ook gewoon ez foutberichten - centrale frontend foutmelding-afhandelaar toegevoegd zodat ik dat ook niet op 10000000000000000 plekken moet doen - kleine validatiebugs in de backend opgelost - de IStep.Validate()-functie doet nu alleen een laatste check of er geen lege verplichte velden meer zijn. De set-service functies doen dat alleen al... ik moet nog even kijken of ik deze functie niet gewoon kan verbannen. De naam is ook wel vaag "Validate" is hier eigenlijk niets meer dan "valideer of er nog lege velden zijn voordat we doorgaan naar de volgende stap". ... dat zou een volgende fix zijn in de architectuur
anna added 12 commits 2026-10-30 16:13:38 +00:00
This pull request doesn't have enough approvals yet. 0 of 1 approvals granted.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin validatie-architectuur-fix:validatie-architectuur-fix
git switch validatie-architectuur-fix
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
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
kyrylo/vorteq-nas-os!229
No description provided.