VNG-Realisatie / huwelijksplanner

Other
4 stars 4 forks source link

Als medewerker wil ik een algemeen login functionaliteit op het dashboard #273

Open CMasselink opened 4 years ago

CMasselink commented 4 years ago

zodat ook testdata niet door iedereen ingezien kan worden.

rubenvdlinde commented 4 years ago

Eens, mij voorstel zou zijn om dan een POC met het user-component te doen. Die staat zowiezo ergens op de backlog en dat voorkomt dubbel werk.

Wat mij betreft zou het er dan puur om gaan dat we in het user component een paar gebruikers zetten, en dat het dashboard deze gebruikt om te valideren. Uiteraard kan zou je op termijn veel meer kunnen met een user component (specifieke rechten etc) maar laat we het zo klein mogelijk houden.

Wat een stretch goal zou zijn is om dan ook meteen de user is en sessie mee te geven aan de NLX headers die het dashboard gebruikt voor component bevragingen. Das niet mega veel extra werk maar vult wel de NLX logs mooi met data die we kunnen checken in de NLX check.

CMasselink commented 4 years ago

@rubenvdlinde wat is er uiteindelijk van het voorstel geïmplementeerd? Is de stretchgoal ook gerealiseerd?

matthiasoliveiro commented 4 years ago

@CMasselink kan je deze hertesten?

CMasselink commented 4 years ago

@matthiasoliveiro @rubenvdlinde Er zit een login op het componenten dashboard dus wat dat betreft akkoord. Ik zou het alleen geen common ground dashboard noemen maar iets van huwelijksplanner componenten dashboard Er zit volgens mij ook een login op het medewerkersportaal maar dat kan ik nu niet testen omdat het medewerkersportaal een foutmelding geeft.

Zou er ook nog naar mijn vraag uit mijn vorige post gekeken kunnen worden? Hoe is het nou gerealiseerd?

.

CMasselink commented 4 years ago

@matthiasoliveiro kleine reminder voor de openstaande vragen

rubenvdlinde commented 4 years ago

ah check, ja dit isgerealiseerd aan de hand van het user component met daarbij inmiddels ook groups en scopes (dit laatste gaat na vandaag ook effect hebben op het dashboard die pakken we mee in het operationeel maken).

Dat leverd een url van een gebruiker op en die wordt vervolgens door de commenground service als nlx-header mee gegeven aan alle calls die worden veroorzaakt door de ingelogde gebruiker.

rubenvdlinde commented 4 years ago

Dit is trouwens gewoon testbaar door in te loggen op het dashboard, er zijn meerdere gebruikers met andere rechten. Allen met hetzelfde test wachtwoord.

balie@utrecht.nl locatie@utrecht.nl trouwambtenaar@utrecht.nl beheer@utrecht.nl

Omdat het hier even gaat om het aantonen van de techniek (er is een seperaat issue voor de inrichting ervan) heeft alleen de beheer gebruiker afwijkende rechten.

rubenvdlinde commented 4 years ago

@bbrands02 kan jij dubbel checken en dan Michelle assignen?

CMasselink commented 4 years ago

Even een reminder @bbrands02

bbrands02 commented 4 years ago

@MichelleDautzenberg Had jij al nieuwe login gegevens ontvangen?

MichelleDautzenberg commented 4 years ago

@bbrands02 Nieuwe logingegevens? Je bedoelt die nieuwe usernames die Ruben hierboven heeft gemeld?