uitwerken volledige bag-foutafhandeling

[fsamwel]

de foutafhandeling feature is uitgebreid zodat deze niet meer afhankelijk is van de foutafhandeling in common. deze combineert de huidige foutafhandeling feature, de foutafhandeling zoals die in common staat en https://github.com/VNG-Realisatie/Haal-Centraal-BAG-bevragen/blob/revert/terugdraaien-geometrie-foutmeldingen/features/foutafhandeling.feature

deze feature beschrijft de foutafhandeling zoals die nu is geïmplementeerd. uitzondering daarop is open issue #510

[strijm]

Deze PR behelst niet het samenvoegen van HC common foutafhandeling.feature en de BAG foutafhandeling.feature uit de branch develop-1.4. Hier wordt weer de hele feature beschrijving gewijzigd zoals ook in het voorstel van Melvin. Dat is niet wat we hebben afgesproken.
Ik kan hier helemaal niets mee, want ik kan niet de verschillen zien t.o.v. de HC common foutafhandeling.feature en de zaken die uit de BAG foutafhandeling.feature komen zijn aangepast naar iets waar ik niet achter sta. Sorry, maar ik kan deze PR onmogelijk goedkeuren.

[fsamwel]

Deze PR behelst niet het samenvoegen van HC common foutafhandeling.feature en de BAG foutafhandeling.feature uit de branch develop-1.4.

@strijm de common beschrijft van alles dat niet in BAG voorkomt, en de BAG API voldoet niet op alle punten aan de foutafhandeling feature in common.
Ik denk dat we hebben afgesproken dat de feature beschrijft hoe het nu werkt, niet hoe de foutafhandeling feature beschrijft hoe het zou moeten werken. Anders moet ik een serie bugs melden die denk ik weinig bijdragen aan de kwaliteit van de API (verschil is redelijk triviaal) en schadelijk kan zijn voor de voortgang.

[fsamwel]

de zaken die uit de BAG foutafhandeling.feature komen zijn aangepast naar iets waar ik niet achter sta.

voor zover ik weet heb ik de zaken uit de huidige BAG foutafhandeling.feature nauwelijks aangepast. ten hoogste de tekst een beetje aangescherpt. Kan je wel aangeven achter welke wijzigingen hier je niet staat en waarom?

[fsamwel]

Hier wordt weer de hele feature beschrijving gewijzigd zoals ook in het voorstel van Melvin.

Ik heb inhoudelijk niks gewijzigd ten opzichte van de common foutafhandeling feature, behalve daar waar de BAG implementatie niet voldoet aan wat in de common foutafhandeling feature staat. In die gevallen beschrijf ik wat de huidige BAG implementatie doet.

Dus het verschil zit puur in de manier van opschrijven, waar elke tabelregel (regels 14-30 en 50-70) uit de common feature is vertaald naar een rule met bijbehorend(e) scenario(s).

[fsamwel]

Ik heb inhoudelijk niks gewijzigd ten opzichte van de common foutafhandeling feature

wat ik wel gedaan heb is toevoegen van verschillende specifieke validaties (zoals min>max) en uitwerken van de paramsCombination situaties

[fsamwel]

@MelvLee en @strijm de feature is aangepast. deze kan weer gereviewd worden

@MelvLee kan jij de test hierop automatiseren, zodat we zeker weten dat dit werkt zoals beschreven?

[strijm]

Zodra de opmerkingen verwerkt zijn, kan deze feature w.m.b. gemerged worden.

[fsamwel]

deze sluiten, in plaats hiervan #523