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

Als developer wil ik het order component door de wasstraat halen #256

Closed
8 of 17 tasks
matthiasoliveiro opened this issue Dec 19, 2019 · 5 comments
Closed
8 of 17 tasks
Assignees
Labels
Feature Request Size M T-shirtsizing for issues Techniek US hoort bij de software developers

Comments

@matthiasoliveiro
Copy link
Collaborator

matthiasoliveiro commented Dec 19, 2019

Zodat het component voldoet aan de common ground normen en internationale standaarden

Voor dit component stellen wij de Schema.org standaard voor de volgende objecten voor:

  • Order
  • OrderItem

Wij stellen het Order-object voor als de logische basis voor dit component, en het OrderItem-object als dependency van het Order-object

Organisatie standaart URL
Schema.org Order https://schema.org/Order
Schema.org OrderItem https://schema.org/OrderItem

Taken lijst todo:

  • Architectuur afstemming
  • Ontwerp gegevens model
  • Realisatie
  • Postman testscripts
  • Documentatie Redoc
  • Testscripts aan leveren aan testvoorziening
  • Voorbeeld implementatie
  • Code opleveren in Utrecht repo

Checklist Definition of done

  • Oas documentatie
  • Postman scripts (test scripts)
  • Aangeleveren testscripts aan de api test voorziening
  • Containers moeten gepubliceert zijn
  • Gereviewd op de design decisions
  • Voorbeeld implementatie beschikbaar
  • Redoc moet beschikbaar zijn
  • Documentatie moet beschikbaar zijn op github-pages
  • NLX integratie moet werken
@matthiasoliveiro matthiasoliveiro added the Techniek US hoort bij de software developers label Dec 19, 2019
@matthiasoliveiro matthiasoliveiro mentioned this issue Dec 19, 2019
10 tasks
@rubenvdlinde rubenvdlinde mentioned this issue Dec 20, 2019
13 tasks
@matthiasoliveiro
Copy link
Collaborator Author

  • Test voorziening geeft niet thuis en kunnen we niet afronden.
  • Github pages is een strech goal en een nice to have, we hebben het nu ander opgelost via de readme (zie tabel)
  • NLX hebben we een andere afspraak voor gemaakt en wordt los opgepakt. Het component is NLX ready maar het staat nog niet aan dat is een andere story

@JanWillemKooi
Copy link
Contributor

Paar vragen:

  • Test voorziening: bedoel je het VNG API Testplatform?
  • Als er geen vinkjes zijn gezet bij ToDo en DoD, betekent dat, dat die dingen nog wel gedaan moeten worden? Anders zou ik ze uit de lijst halen.
  • De opmerking over NLX houdt dit issue niet tegen om naar Done te gaan?

@CMasselink
Copy link
Collaborator

Het datamodel dat je via redoc kan openen, klopt niet. lijkt het datamodel van het PDC te zijn. Order en order item komen er niet op voor.

Bij orderitem zit een description. Hoe verhoudt dit attribuut tot description bij offer & bij product? Moet de description van een orderitem niet gewoon van een product overgenomen worden en niet nog een keer gedupliceerd bij orderitem?
afbeelding

Foute link verwijzingen:
afbeelding

@rjzondervan
Copy link
Collaborator

Hoi Constantijn,

Het datamodel van het ordercomponent staat op dit moment nog in de dev-versie. De dev-versie van het component is alleen op dit moment niet gedeployed, dus hierbij een link naar het juiste datamodel: https://github.com/ConductionNL/orderregistratiecomponent/blob/development/api/public/schema/datamodel.pdf
Als dit model akkoord is kunnen we het beschikbaar maken in de productieversie.

De description in OrderItem is verder een optioneel veld dat is meegenomen omdat we de afspraak hanteren dat in principe elke entiteit minimaal een name en description heeft.

@rubenvdlinde rubenvdlinde added Feature Request Size M T-shirtsizing for issues labels Feb 13, 2020
@rjzondervan rjzondervan assigned CMasselink and unassigned rjzondervan Mar 9, 2020
@CMasselink
Copy link
Collaborator

Akkoord. Met als generiek restpunt de URL verwijzingen naar de repository en docker container. En de foutieve URL verwijzingen in de voorbeeld URLs (zijn al andere userstory's van)
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Request Size M T-shirtsizing for issues Techniek US hoort bij de software developers
Projects
None yet
Development

No branches or pull requests

5 participants