Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SDIMEV-41: REV-helpdesk 22 01 2718: invullen getal aantalBevoorradingen als enumeratie #39

Closed
PB-GNM opened this issue Jan 27, 2022 · 10 comments
Assignees
Labels
enhancement New feature or request oplossingsvoorstel v1.3 verwerkt in versie 1.3 van IMEV

Comments

@PB-GNM
Copy link
Collaborator

PB-GNM commented Jan 27, 2022

Dit is een wijzigingsverzoek uit de REV-helpdesk (22 01 2718 ) die als volgt is omschreven "invullen getal aantalBevoorradingen (A7: OpslagPropaanPropeen_VasteAfstandGeenVergunningplicht) aanpassen naar categorie (enumeratie)"

@PB-GNM
Copy link
Collaborator Author

PB-GNM commented Jan 27, 2022

Welke waarden mogen er dan in de enumeratie?

@PB-GNM PB-GNM changed the title SDIMEV-41: invullen getal aantalBevoorradingen SDIMEV-41: invullen getal aantalBevoorradingen als enumeratie Jan 27, 2022
@PB-GNM PB-GNM changed the title SDIMEV-41: invullen getal aantalBevoorradingen als enumeratie SDIMEV-41: REV-helpdesk 22 01 2718: invullen getal aantalBevoorradingen als enumeratie Jan 27, 2022
@manuelbeterams
Copy link

Voor het aantal bevoorradingen staan in het Bal tabel 4.899 twee opties, 5 of minder bevoorradingen of meer dan 5.

@PB-GNM PB-GNM added the enhancement New feature or request label Feb 2, 2022
@PB-GNM
Copy link
Collaborator Author

PB-GNM commented Feb 9, 2022

Op de plek waar nu "Integer" staat, komt dan CategorieAantalBevoorradingen, en dat verwijst dan naar een enumeratie met de 2 waarden zoals hierboven voorgesteld.

image

@PalmJanssen
Copy link
Contributor

Werkgroep:
Bal tabel 4.899.
Noemt twee categorien <= 5 en > 5.
Het is de vraag of we liever de indiener een schatting laten maken of dat we de indiener een categorie laten invullen. (actie Peter)

De Opslag inhoud wordt ook in Tabel 4.899 genoemd maar zit bij ReferentieEVContour bij OpslagReferentie.inhoud

@PB-GNM
Copy link
Collaborator Author

PB-GNM commented Aug 24, 2022

Inmiddels is Peter vervangen door Frans Geurts.
Frans geeft aan dat het vooral gewenst is omdat er dan minder fouten gemaakt kunnen worden door de indieners.
Er is een relatie met #40.

@PB-GNM PB-GNM closed this as completed Aug 24, 2022
@PB-GNM PB-GNM reopened this Aug 24, 2022
@PB-GNM
Copy link
Collaborator Author

PB-GNM commented Aug 24, 2022

Het oplossingsvoorstel is om de enumeratie in te voeren.

@PB-GNM
Copy link
Collaborator Author

PB-GNM commented Aug 31, 2022

Aanleiding wijziging

De aanleiding is dat er minder fouten worden gemaakt wanneer er categorieën gebruikt worden.

Voorgestelde wijziging

Het voorstel is om op de plek waar nu "Integer" staat de enumeratie CategorieAantalBevoorradingen, en dat verwijst dan naar een enumeratie met de 2 waarden "<= 5" en "> 5".

image

Impactanalyse

Geef hier een indicatie van de impact van de wijziging:

  • Wie gaat er wat van merken? dataprovider en gebruiker REV en Softwareleveranciers
  • Verandert de semantiek van de objecten? Nee
  • Heeft het impact op het json-schema? Ja

Prioriteit

Middel

Toelichting

Er is een relatie met #40.

@PB-GNM
Copy link
Collaborator Author

PB-GNM commented Sep 28, 2022

Expertgroep:
Aandachtspunt. Er is een extra conversie nodig en dat is een risico.
Nieuw voorstel: keuze object: categorie of exact aantal.

@PalmJanssen PalmJanssen added the v1.3 verwerkt in versie 1.3 van IMEV label Nov 18, 2022
@PalmJanssen
Copy link
Contributor

verwerkt:
image

@PB-GNM
Copy link
Collaborator Author

PB-GNM commented Jan 30, 2023

Opgelost in versie 1.3.

@PB-GNM PB-GNM closed this as completed Jan 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request oplossingsvoorstel v1.3 verwerkt in versie 1.3 van IMEV
Projects
None yet
Development

No branches or pull requests

3 participants