Feature/raymond/gemeentelijk vastgoed v2

[raymondyoung2016]

Hallo collega's,
Ik heb op github een pullrequest aangemmaakt vanuit mijn featurebranch voor een nieuwe dataset gemeentelijk_vastgoed met daarin 3 tabellen:

• master...feature/raymond/gemeentelijk_vastgoed_v2
  Naar de master. Mijn collega's in het team zijn akkoord.
  Dit request is voor de merge naar master, en uitrol in RefDB zo mogelijk voorlopig alleen in ACC.
  Bevat breaking changes: nieuwe set en tabellen.

[raymondyoung2016]

Het zou heel fijn zijn als een van jullie dit nog zou kunnen reviewen, goedkeuren en mergen, zodat we hiermee verder kunnen met ontwikkelen en testen in ACC. Ik weet niet of ik nog expliciet iemand had moeten vragen om te reviewen, misschien Yashar / @Yrob? Bij voorbaat dank!

[raymondyoung2016]

Hallo Barry, ik heb featurebranch aangepast: Terugdraaien toev. gemeentelijk_vastgoed aan datasets.json. Kan je nog eens reviewen?

[raymondyoung2016]

Ik heb in alle drie de tabellen deze fout verbeterd: Gem. Vastgoed v2: verbet. required en display. Kan je nog eens reviewen?

[raymondyoung2016]

En nu: Gem. Vastg. v2: verbet. veld Identif: obj/vhe-id

[raymondyoung2016]

Barry Daniels
Ik moet er vandoor, je kunt hier kijken of de pre-commit goed gaat:
https://github.com/Amsterdam/amsterdam-schema/actions/runs/12417203616/job/34667617927?pr=1214
Ik kan morgen eventueel nog even kijken om hem te mergen.

Raymond Young
Nee, gaat niet goed maar ik zie niet waarom.
Dank voor hulp.
Kan jij eventueel kijken wat er misgaat en dat verbeteren?

[raymondyoung2016]

Hallo Barry Daniëls,
ik weet niet zeker of ik hier nog aan toe ga komen, maar misschien kunnen mijn collega-ontwikkelaars (Tino Cox, Sam van Oort, Djoeke Doesburg of Pieter Smit) je opmerkingen verwerken? Allen dank daarvoor!

[raymondyoung2016]

Weet jij al meer over wat hier misgaat, Barry of collega's, ga je hier nog aan toe komen? Ik hoop dat jullie dit nog kunnen mergen en in ACC uitrollen.

[Khelied]

iom Raymond. PR gesloten