-
Notifications
You must be signed in to change notification settings - Fork 4
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
Wastraat #263
Comments
@rjzondervan Hoewel het wasstraat verhaal inmiddels geweest is kunnen we dit issue nu even aanhouden voor het bijwerken van de datamodellen op de verschillende repositories |
@rubenvdlinde bijwerken datamodellen zijn review/rest punten van de overige userstory's toch? |
Yup, en dat lopen we per component als onderdeel van de DOD bij deployen. Los vand at we daar an zich wat strakker op mogen zitten kan dit koepel issue wat mij betreft dan ook eigenlijk weg. Het oorspronkelijke doel (alle componenten bijwerken) is immers inmiddels gedaan. Ze beschikken daarmee dan ook over de benodigde code, er zijn losse issues om e.a. nog te configureren en aan te zetten (die heb ik gisteren geprobeerd inzichtelijk te maken) maar dat kunnen we mee nemen in de deploy die we moeten doen om alle componenten van auth te voorzien (dus vanmiddag of morgen ochtend). Wat mij betreft zouden we dit issue dan ook kunnen sluiten en eventuele vervolg vragen los inschieten. |
Volgens mij kunnen we deze afsluiten maar even een dubbel check @CMasselink |
Als developer wil ik een aantal componenten door de Wastraat zodat de code up te date is en ze de laatste commonground functionaliteit beschikbaar is.
Hiervoor moeten we doen:
Hierdoor word de volgende functionaliteit toegevoegd
Dit moet gebeuren voor de volgende componenten
The text was updated successfully, but these errors were encountered: