search

VNG-Realisatie / api-test-platform

6 stars 3 forks source link

Supplier information kan onterecht per run worden aangepast #325

Closed HenriKorver closed 4 years ago

HenriKorver commented 4 years ago

Bug

Supplier information kan nu per run worden aangepast. Je zou zeggen dat je de supplier information slechts op environment-niveau zou mogen wijzigen.

alextreme commented 4 years ago

Toevallig is dit een aanpassing dat @stevenbal al binnen een andere issue heeft verholpen. Ik weet nog niet of dit al op staging staat maar ik heb zijn wijzigingen goedgekeurd.

stevenbal commented 4 years ago

Op staging kan de supplier informatie nu alleen op environment niveau aangepast worden: https://staging.api-test.nl/server/2/059e9c61-e832-48af-8ad1-5b2bcc312b6b/09d84481-a1a0-4581-8002-1209d85bf97b/ Screenshot from 2019-12-03 10-01-05

Deze informatie is ook nog weergegeven bij de individuele runs voor dat environment: https://staging.api-test.nl/server/2/9c4a95c3-3166-43fa-8448-298cd7b0fac6/ Screenshot from 2019-12-03 10-01-02

En daar kan alleen de software version aangepast worden. Ik dacht dat het wel handig zou zijn om de supplier informatie ook op de detail pagina's weer te geven, maar ik kan ze beter loshalen van elkaar denk ik, dus een apart tabelletje voor de supplier informatie van het environment, en een andere voor de software versie @HenriKorver

HenriKorver commented 4 years ago

Ik begrijp niet waarom er onderscheid wordt gemaakt voor het veld "software version". Volgens mij wil je dat veld ook niet per run kunnen aanpassen. Een nieuwe software versie betekent in mijn ogen een nieuwe environment

stevenbal commented 4 years ago

Dat is gedaan naar aanleiding van deze https://github.com/VNG-Realisatie/api-test-platform/issues/316 user story. Het is nu ook zo dat de referentieimplementatie geupdate wordt, maar met hetzelfde environment getest wordt toch?

HenriKorver commented 4 years ago

Ja je hebt gelijk. Alleen bij een major software update waarbij ook de URL van de API veranderd zal je een nieuwe environment moeten aanmaken, maar dat gaat vanzelf goed.

HenriKorver commented 4 years ago

Ik dacht dat het wel handig zou zijn om de supplier informatie ook op de detail pagina's weer te geven, maar ik kan ze beter loshalen van elkaar denk ik, dus een apart tabelletje voor de supplier informatie van het environment, en een andere voor de software versie

Zo als het nu is vind ik het eigenlijk wel prima. Dus gewoon alle supplier gegevens bij elkaar houden inclusief software version en ervoor zorgen dat je per run alleen de software version kunt aanpassen.

alextreme commented 4 years ago

Ter controle door @HenriKorver op staging, wel nog met een opmerking van mijn kant over de UI maar die kan Steven oppakken in #316

HenriKorver commented 4 years ago

Als je op run-niveau de "software version" wilt aanpassen krijg je in het update scherm de term "build version". Graag op alle plekken dezelfde naam gebruiken voor hetzelfde veld. Anders kan het leiden tot verwarring.