# Verslag vergadering 8
Locatie: online (Discord)

Datum: 18/04/2021 14u30

Aanwezigen: Jan-Pieter, Iwijn, Ewout, Freya, Gertjan, Chato, Victor, Elias

Verontschuldigd: ---

Afwezigen: ---

Notulist: Jan-Pieter

# Agendapunten:
1. Algemeen
- Goedkeuring [vorig verslag](https://github.ugent.be/SoftwareEngineeringLab2/2020-2021-groep-01/pull/562)
- Deadlines sinds vorige vergadering
- Documenten updaten
- [ ] Klassendiagram (Gertjan en Jan-Pieter)
- Wacht nog tot de backend af is
- [ ] Uitbreidingen (Ewout en Iwijn)
- [ ] Demo met voice-over (Elias)
- [ ] Gebruikershandleiding (Chato en Gertjan)
- Wachten op nieuwe layout van Victor
- [ ] Use cases (Victor)
- [x] Use cases checken voor proposal home page view voor magda medewerker (Iwijn)
- nummering moet aan het einde gecheckt worden
- [x] Component overview (Freya)
- PSA: Zet een description bij je issue, al is het kort, zet het erbij want issues zijn soms wat onduidelijk
- Itsme beslissing
- Presentatie
- Wie wil presenteren
- Jan-Pieter: wil gerust over backend spreken
- Gertjan : wil ook gerust iets zeggen
- Ewout: wil gerust ook weer spreken als ik de 3de presentatie niet moet doen
- Elias weer demo? (als hij wilt)
- Wat moeten we bespreken
- Status:
- Algemeen:
- misschien eens ms2 overlopen zodat we niets vergeten zijn
- document nodig voor afwijkingen van api, waaronder:
- out of scope serviceProvider veld bij org
- geen lijst van bronnen mogelijk ookal hebben we dat gemeld
- member endpoint is vrij nutteloos
- contacts endpoint is vrij nutteloos
- deprecated veld bij een service hoort eigenlijk bij een serviceInstance
- config endpiont is vrij nutteloos
- bij demo melden wat we nog gaan doen:
- paging
- betere foutmeldingen (zie hieronder)
- Frontend:
- moet proposal extra velden krijgen?
- fouten en vinkjes bij aanvraag: moeten duidelijke foutmeldingen krijgen, vinkjes moeten hier van afhangen en ook afhangen van oppervlakkige controles - als dit nog niet is tegen ms2 zou ik dit zeker vermelden in de demo dat we dat nog gaan doen
- nieuwe layout
- Moet nog gebeuren
- goedkeuren proposal
- bewerken proposal
- contacten toevoegen aan organisatie
- certificaten in proposalflow
- volledige flow checken
- Backend:
- UUIDs ipv longs bij alle entities (db-breaking change)
- Meeste issues zijn opgelost, nog wat kleine dingen te doen
- Authutils bijna af
- Enkel nog Files endpoint
- Wat willen we hier?
- Voorstel: alle ingelogde users mogen uploaden en lezen, enkel magda-admin mag verwijderen
- CSR hoeft enkel nog wat tests te krijgen
- Documenten linken bij proposal/permission moet nog gebeuren
- Contacts endpoint, wat doen we ermee? Belangrijk voor front end
- Zie puntje bij API beheerder
- We wijken af van de API spec, maar dat is besproken met frontend
- Issues te bespreken:
- Issue 625: klant kan aan alle users
- Dit willen we eigenlijk niet
- Dit impacteert de endpoints om user toe te voegen aan organisatie, maar kunnen we oplossen door email mee te geven ipv id
- Idee: organisaties > klik op organisatie > klik voeg user toe > je krijgt veld om user email in te vullen > bestaat = toegevoegd, bestaat niet = invite link (evt voor MS3)
- Issue 621: had veel vroeger gecommuniceerd moeten worden
- User die niet verified is? wat doen we
- Documentatie:
- Uitbreidingen bijna af, nog een paar kleine suggestions toe te voegen
- Review zo snel mogelijk, zodat we dit bestand niet moeten blijven aanpassen

- Voorstel version nummering:
- we gebruiken x.y.z version numbers
- bij iedere PR voor kleine bugfix doen we z+1
- bij iedere PR voor feature doen we y+1 en z=0
- bij iedere Weekly release x+1 en y=z=0
- noem de commit altijd "Version bump" en doe die vlak voor het mergen
- Bij documentatie is (natuurlijk) geen version bump nodig
- File noemt VERSION
2. Projectleider
- Statusupdate?
3. Systeembeheerder
- Statusupdate?
4. API beheerder
- Statusupdate?
- Members endpoint lijkt me nogal redundant, ik stel voor dat members gewoon de lijst van users binnen de organisatie geven, en dan een filter op member/admin
- Wat is die `/config` ?
- Contacts endpoint lijkt me ook nogal nutteloos, een lijst van emails+phonenumber die we in de backend linken (of aanmaken) aan users lijkt me beter om eerlijk te zijn ~Jan-Pieter
- Contacts hebben ook nog een rol binnen een aanvraag, hoe doen we dat dan, in de proposal opslaan lijkt mij het logische antwoord
- Een contact is toch binnen een organisatie? maar contactpersonen zou ik als map van users->rol opslaan in een proposal (en die kunnen we dan via de token access geven tot een specifieke proposal via de id)
5. Test beheerder
- Statusupdate?
- Frontend testen status?
6. Documentatiebeheerder
- Statusupdate?
7. Customer Relations Officer
- Statusupdate?
8. Secretaris
- Statusupdate?
9. Human Relations Officer
- Statusupdate?
10. Datum volgende vergadering:
- Voorstel: 21/04 om 19u30
11. Varia
- Ik ben kapot van de stress, kunnen we voor MS3 please de interne deadline om alles werkende te hebben 1 week voor de deadline ofzo? want nu ben ik echt gewoon kapot en ben ik het beu. Als we de interne deadline niet doen leg ik voor mezelf een datum op waarna ik niks meer doe aan het project dat ik niet kapot ga van de stress ~Jan-Pieter

# Verslag:
## 1. Algemeen
- Goedkeuring [vorig verslag](https://github.ugent.be/SoftwareEngineeringLab2/2020-2021-groep-01/pull/562)
- Deadlines sinds vorige vergadering
- Documenten updaten
- [ ] Klassendiagram (Gertjan en Jan-Pieter)
- Wacht nog tot de backend af is
- [ ] Uitbreidingen (Ewout en Iwijn)
- Bijna af, nog een aantal suggesties op te lossen
- Standaard taal nederlands
- Header ipv region detection
- [ ] Demo met voice-over (Elias)
- Nog wachten tot frontend feature complete is, liefst maandag al, maar kan mogelijks pas dinsdag
- [ ] Gebruikershandleiding (Chato en Gertjan)
- Wachten op nieuwe layout van Victor
- Vandaag af
- Nieuwe screenshot
- Flow verandert niet
- [ ] Use cases (Victor)
- [x] Use cases checken voor proposal home page view voor magda medewerker (Iwijn)
- nummering moet aan het einde gecheckt worden
- Freya gaat er ook nog naar kijken
- Victor is al begonnen om na te lezen
- Auth flow moet correcter
- Details mogen er nog bij maar niet te veel
- Genoeg interactie met systeem en gebruiker
- [x] Component overview (Freya)
- Is af en merged
- PSA: Zet een description bij je issue, al is het kort, zet het erbij want issues zijn soms wat onduidelijk
- Itsme beslissing
- We willen het sowieso bekijken
- Ze willen het echt graag, dus ik zou ervoor kijken
- Team is unaniem voor
- Iedereen wil eraan meehelpen
- Presentatie
- Wie wil presenteren
- Jan-Pieter: wil gerust over backend spreken
- Gertjan: wil ook gerust iets zeggen
- Ewout: wil gerust ook weer spreken als ik de 3de presentatie niet moet doen
- Elias: weer demo
- Victor: iets over frontend
- Wat moeten we bespreken
- Bij demo zeggen wat er nog gaat gebeuren
- 2e persoon die zegt wat er nog bijkomt
- Features
- Toegevoegd
- Auth
- Files uploaden
- Toekomstige
- Nieuwe gebruikte technologieën
- JWT
- Mails
- CSR
- Ontwerpbeslissingen
- Ervaringen
- Stappen richting volgende milestone
- Effectief gedaan
- Nieuwe te doen
- Namen van iedereen
- Deadline:
- Dinsdagavond 21u
- Presentatie timing:
- 11u15-11u45
- Peer reviews reminder
- Tegen dinsdagavond, niet vergeten!
- Status:
- Algemeen:
- misschien eens ms2 overlopen zodat we niets vergeten zijn
- Aanvraag documenten
- Certificaten bijna af
- Interactie en sequence diagrammen (Gertjan)
- document nodig voor afwijkingen van api, waaronder:
- md file in github bij documentation
- schema `| api versie | onze versie | reden |`
- out of scope: serviceProvider veld bij org
- geen lijst van bronnen mogelijk ookal hebben we dat gemeld
- Is gemeld van magda uit dat het moet, dus dat is de reden
- member endpoint is vrij nutteloos
- contacts endpoint is vrij nutteloos
- deprecated veld bij een service hoort eigenlijk bij een serviceInstance
- Doen we wel volgens API
- config endpoint is vrij nutteloos
- `/users/{id}/organisations` blijft bij ons
- POST/PATCH `/users/{id}/organisations/{orgid}&admin=true/false` toevoegen aan onze backend
- certificates
- DELETE naar certificate by id doen we niet
- uitbreidingen proposal
- UUID ipv Long
- bij demo melden wat we nog gaan doen:
- paging (nice to have)
- betere foutmeldingen frontend (zie hieronder)
- Frontend:
- moet proposal extra velden krijgen?
- Duidelijk maken voor backend welke het moeten zijn
- fouten en vinkjes bij aanvraag: moeten duidelijke foutmeldingen krijgen, vinkjes moeten hier van afhangen en ook afhangen van oppervlakkige controles - als dit nog niet is tegen ms2 zou ik dit zeker vermelden in de demo dat we dat nog gaan doen
- Controle in frontend voor er checkmarks komen
- nieuwe layout
- Victor toont tijdens de vergadering de nieuwe layout
- Ziet er goed uit voor iedereen
- Moet nog gebeuren
- goedkeuren proposal
- bewerken proposal
- status proposal tonen
- contacten toevoegen aan organisatie
- certificaten in proposalflow
- volledige flow checken
- Backend:
- UUIDs ipv longs bij alle entities (db-breaking change)
- Meeste issues zijn opgelost, nog wat kleine dingen te doen
- Authutils bijna af
- Enkel nog Files endpoint
- Wat willen we hier?
- Voorstel: alle ingelogde users mogen uploaden en lezen, enkel magda-admin mag verwijderen
- CSR hoeft enkel nog wat tests te krijgen
- Documenten linken bij proposal/permission moet nog gebeuren
- Contacts endpoint, wat doen we ermee? Belangrijk voor front end
- Zie puntje bij API beheerder
- We wijken af van de API spec, maar dat is besproken met frontend
- Issues te bespreken:
- Issue 625: klant kan aan alle users
- Dit willen we eigenlijk niet
- Dit impacteert de endpoints om user toe te voegen aan organisatie, maar kunnen we oplossen door email mee te geven ipv id
- Idee: organisaties > klik op organisatie > klik voeg user toe > je krijgt veld om user email in te vullen > bestaat = toegevoegd, bestaat niet = invite link (evt voor MS3)
- Issue 621: had veel vroeger gecommuniceerd moeten worden
- Om dat te voorkomen, post in #general
- User die niet verified is? wat doen we
- Mogen inloggen? Nee, maar moet dan specifieke error code geven
- Pas voor MS3
- Documentatie:
- Uitbreidingen bijna af, nog een paar kleine suggestions toe te voegen
- Review zo snel mogelijk, zodat we dit bestand niet moeten blijven aanpassen
- Status andere bestanden?
- Voorstel version nummering:
- we gebruiken x.y.z version numbers
- bij iedere PR voor kleine bugfix doen we z+1
- bij iedere PR voor feature doen we y+1 en z=0
- bij iedere Weekly release x+1 en y=z=0

## 2. Projectleider
- Statusupdate?

## 3. Systeembeheerder
- Statusupdate?
- Als je opnieuw backend deployt, opletten voor de db-breaking changes

## 4. API beheerder
- Statusupdate?
- Members endpoint lijkt me nogal redundant, ik stel voor dat members gewoon de lijst van users binnen de organisatie geven, en dan een filter op member/admin
- Wat is die `/config` ?
- Contacts endpoint lijkt me ook nogal nutteloos, een lijst van user entiteit objecten die we in de backend linken (of aanmaken) aan users lijkt me beter om eerlijk te zijn ~Jan-Pieter
- Contacts hebben ook nog een rol binnen een aanvraag, hoe doen we dat dan, in de proposal opslaan lijkt mij het logische antwoord
- Een contact is toch binnen een organisatie? maar contactpersonen zou ik als map van rol->users opslaan in een proposal (en die kunnen we dan via de token access geven tot een specifieke proposal via de id)

## 5. Test beheerder
- Statusupdate?
- Frontend - sonarqube link zal pas later gebeuren want het werkt nog niet
- Frontend testen status?
- zit rond de 30%

## 6. Documentatiebeheerder
- Statusupdate?

## 7. Customer Relations Officer
- Statusupdate?
- Mail?
- nog geen antwoord

## 8. Secretaris
- Statusupdate?

## 9. Human Relations Officer
- Statusupdate?

## 10. Datum volgende vergadering:
- Voorstel: 21/04 om 14u

## 11. Varia
- Ik ben kapot van de stress, kunnen we voor MS3 please de interne deadline om alles werkende te hebben 1 week voor de deadline ofzo? want nu ben ik echt gewoon kapot en ben ik het beu. Als we de interne deadline niet doen leg ik voor mezelf een datum op waarna ik niks meer doe aan het project dat ik niet kapot ga van de stress ~Jan-Pieter
- Frontend veel eerder backend linken
- Interne deadline voor alles behalve "nice to have" issues
- 1 week op voorhand?
- 09/05 middernacht
- Documenten erna

# Taakverdeling:
| Taak | Verantwoordelijke | Deadline |
| ---- | ----------------- | -------- |
| Mail sturen ivm itsme beslissing | Chato | 21/04/2021 |

# Datum volgende vergadering
21/04 om 14u