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

rolomschrijvingGeneriek verwijderd #296

Merged
merged 1 commit into from
Aug 16, 2018

Conversation

sergei-maertens
Copy link
Collaborator

Wijzigingen

Browsable API spec

het veld rolomschrijvingGeneriek is verwijderd uit de API:

  • er is geen expliciete vraag naar (vraag moet uit user stories komen)
  • het is onduidelijk wat het verschil is met het veld rolomschrijving

We stellen dan ook een wijziging voor in het RGBZ om dit attribuut op een rol te verwijderen, bij gebrek aan praktische requirements voor dit veld/attribuut.

Merk ook op:

  • rolomschrijving heeft een waardenverzameling (enum), gedefinieerd als AN80
  • rolomschrijvingGeneriek heeft de waardenverzameling van rolomschrijving + media-initiator, gedefnieerd als AN40
  • ImZTC heeft rolomschrijvingGeneriek met de waardenverzameling van rolomschrijving, gedefinieerd als AN20.

De consistentie is hier eigenlijk helemaal zoek, en die trekken we recht in deze API specs.

Copy link
Collaborator

@michielverhoef michielverhoef left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Conform SCRUM alleen dat opnemen wat nodig is voor de userstory.

* (nog) geen expliciete vraag naar
* betekenisverschil met rolomschrijving is niet helder - is dit in de
  praktijk uberhaupt nodig?
@sergei-maertens sergei-maertens merged commit d8b0216 into master Aug 16, 2018
@sergei-maertens sergei-maertens deleted the feature/us-45-zet-behandelaar branch August 27, 2018 12:09
@ArjanKloosterboer
Copy link
Contributor

ArjanKloosterboer commented Aug 27, 2018

het is onduidelijk wat het verschil is met het veld rolomschrijving

Definitie Rolomschrijving: "Omschrijving van de aard van de ROL". Definitie Rolomschrijving_generiek: "Algemeen gehanteerde benaming van de aard van de ROL". Zie verder de toelichtingen bij beide attribuutsoorten.

rolomschrijvingGeneriek heeft de waardenverzameling van rolomschrijving + media-initiator, gedefnieerd als AN40

Rolomschrijving heeft geen waardeverzameling. Rolomschrijving_generiek heeft daarentegen wel een enumeratie en is derhalve niet AN40.

ImZTC heeft rolomschrijvingGeneriek met de waardenverzameling van rolomschrijving, gedefinieerd als AN20

ImZTC kent Roltypeomschrijving (AN20) en Roltypeomschrijving_generiek (met enumeratie). Dit is geheel overeenkomstig RGBZ2.

De consistentie is hier eigenlijk helemaal zoek, en die trekken we recht in deze API specs.

Er was en is consistentie in RGBZ2 en in ImZTC en tussen deze infomodellen.

@sergei-maertens, graag dit weer rechttrekken in de resource d.w.z. Rolomschrijving_generiek weer toevoegen.

Copy link
Contributor

@ArjanKloosterboer ArjanKloosterboer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rolomschrijving heeft geen enumeratie. Rolomschrijving_generiek daarentegen wel. Niet akkoord dus!

@ArjanKloosterboer
Copy link
Contributor

Punt van onderzoek is of alle velden die nu van ROL verplicht zijn in het RGBZ terecht verplicht zijn. Onderzoek ik.

@TCIMEddy
Copy link
Contributor

@ArjanKloosterboer Mooi! kunnen ze dan verder voor nu of moet het onderzoek afgewacht worden (meer procesvraag)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants