nl-digigo / visi

Beheeromgeving van de VISI open standaard.
https://nl-digigo.github.io/visi/
6 stars 4 forks source link

Validatie van berichten met 1.6 raamwerk obv huidige promotor / nieuwe manier van valideren #87

Open JanaxLooij opened 4 years ago

JanaxLooij commented 4 years ago
  1. Omschrijving van de vraag/opmerking zelf (bij voorkeur in het format van ‘userstory’):

“Als VISI gebruiker wil ik dat berichten gevalideerd kunnen worden zodat ik zeker weet dat ik valide berichten conform het raamwerk verzend en ontvang”

"Als VISI leverancier wil ik een goede manier van valideren omdat de huidige technische implementatie verouderd is en niet makkelijk aan te passen"

  1. Stakeholders: Iedere VISI gebruiker / iedere VISI leverancier

  2. Datum en oorsprong verzoek: 08-11-2019, VISI EC werksessie: nav ontbrekende onderdelen uit 1.6 in huidige promotor

  3. Waarop heeft de vraag betrekking: a. Open Standaard (VISI) b. VISI-raamwerk c. Software (VISI)

  4. Wat is de urgentie: ‘S’ (should have; hinder kan worden omzeild in dagelijks werk; moet binnen redelijke termijn worden opgelost)

  5. Toelichting op de urgentie: Wanneer gesteld kan worden dat validatie van berichten verplicht is moet dit zsm opgelost worden voor systematiek versie 1.6

User story voor het eerst gemeld in 2009 https://github.com/bimloket/visi/issues/10

‘Open Standaard’

  1. Stel vast of het een ‘bug’ is, of een ‘verbetervoorstel’, of een ‘ander verzoek’

Deels bug (validatie werkt nu niet goed met 1.6), deels verbetervoorstel want huidige manier van validatie via door promotor gegenereerde xsd kan op andere manieren beter en uitgebreider

  1. Bug

a. Beschrijf het bestaande gedrag Zie https://github.com/bimloket/visi/issues/84

b. Beschrijf het gewenste gedrag Volledig correcte validatie obv 1.6

c. Benoem de stappen om het probleem te reproduceren Genereer een xsd (met de promotor) van het nieuwste _7.xml raamwerk https://github.com/bimloket/visi/blob/master/testproject/topkoks/_7.xml Controleer of er gevalideerd wordt op de punten die beschreven worden in: https://github.com/bimloket/visi/issues/84

  1. Verbetervoorstel

a. Maak de userstory expliciet; mogelijk valt die uiteen in meerdere user stories Een betere manier van valideren, zoals bijvoorbeeld beschreven in: https://github.com/bimloket/visi/issues/14 (Embedded validator in VISI software) Zie ook excel blad: https://github.com/bimloket/visi/files/2970376/kruistabel.-.validatie.van.berichten-v3.xlsx

b. Beschrijf de randvoorwaarden Snellere validatie, makkelijk aan te passen in de toekomst (dwz nieuwe systematiek versies moeten gemakkelijk verwerkt worden in de manier van valideren)

c. Completeer de ‘definiton of ‘done’ Volledig correcte validatie

d. Beschrijf het verbetervoorstel / of gewenst gedrag Nieuwe manier van valideren of bijgewerkte promotor

e. Beschrijf de voordelen van de verbetering Betere manier van valideren, volledig correcte validatie obv 1.6

f. Beschrijf eventuele “Alternate Flows” Geen validatie, pen en papier

g. Beschrijf eventuele “Exception Flows”
Validator (of xsd via promotor) levert geen correcte en volledige validatie op

image

niekpluijmert commented 4 years ago

Kan door de EC worden opgepakt

niekpluijmert commented 4 years ago

onterecht afgesloten

ArneBruinse commented 3 years ago

zie ook deze: https://github.com/bimloket/visi/issues/55

ArneBruinse commented 3 years ago

Zie ook us #110

ArneBruinse commented 2 years ago

@ElisabethKloren @niekpluijmert Graag deze op de agenda van de vergadering van 18-2-2022 van de VISI expertcommisie alsjeblieft. We moeten goed bespreken wat de expertcommissie wel en niet kan betekenen voor dit punt.

ArneBruinse commented 2 years ago

@ElisabethKloren voor de notulen: we hebben besproken dat het ons als expertcommissie zinvol lijkt om te overwegen als speerpunt voor de 1.8 om een oplossing te vinden om los te komen van de huidige promotor.

Wel moet eerst voor de 1.7 #110 afgerond worden om tot een goede probleemstelling te komen. Deze kan dan vervolgens ook gebruikt worden in de onderbouwing zodra we dit issue dan officieel willen voordragen.