Larsvandijk / WT_backend

0 stars 1 forks source link

UML klassendiagram #9

Open Larsvandijk opened 3 months ago

Larsvandijk commented 3 months ago

https://app.diagrams.net/#G1UdyktBpWN9ZrQX6DwVyHn1JmTcO01R9V#%7B"pageId"%3A"aazOFcrTyHrF6c7-vA-d"%7D

TheAaltje23 commented 3 months ago

@Jopvdhoef zou jij hier naar kunnen kijken? Reviewen, eigen input geven etc.

HoangBuiGit commented 3 months ago

The class diagram looks good! It is a great overview of what needs to be done. We (Tom and me) do have a few questions.

Jopvdhoef commented 3 months ago

Ziet er goed uit. Aantal vraagjes:

iliendo commented 3 months ago

Ziet er goed uit. Aantal vraagjes:

  • Moet er niet nog een soort anonymous user role bij?
  • Moeten de rollen Trainer, Trainee en Employee echt aparte classes worden? Ik ben nu een beetje aan het kijken hoe dit soort dingen geimplementeerd worden in Spring en ik zie nu een voorbeeld waar de role gwn wordt opgeslagen als Enum in de user class.

Thanks voor de feedback!

Wij wisten niet zo goed hoe we de anonymous user moesten toevoegen aan het klassendiagram (en of die toegevoegd moet worden). Dat is een vraag die wij aan Martijn gaan stellen vrijdag.

Wat betreft de rollen dachten wij dat elke rol andere functionaliteit zou moeten hebben. Dus een trainer zou bijvoorbeeld een boek mogen toevoegen/verwijderen. Daarom hebben wij ervoor gekozen om aparte klassen aan te maken.

kiknl commented 3 months ago

Ziet er goed uit. Aantal vraagjes:

  • Moet er niet nog een soort anonymous user role bij?
  • Moeten de rollen Trainer, Trainee en Employee echt aparte classes worden? Ik ben nu een beetje aan het kijken hoe dit soort dingen geimplementeerd worden in Spring en ik zie nu een voorbeeld waar de role gwn wordt opgeslagen als Enum in de user class.

Thanks voor de feedback!

Wij wisten niet zo goed hoe we de anonymous user moesten toevoegen aan het klassendiagram (en of die toegevoegd moet worden). Dat is een vraag die wij aan Martijn gaan stellen vrijdag.

Wat betreft de rollen dachten wij dat elke rol andere functionaliteit zou moeten hebben. Dus een trainer zou bijvoorbeeld een boek mogen toevoegen/verwijderen. Daarom hebben wij ervoor gekozen om aparte klassen aan te maken.

Ik ben het er zelf ook mee eens dat we verschillende rollen moeten hebben om zo andere functionaliteiten en restricties per rol toe te wijzen.

kiknl commented 3 months ago

We moeten ook de terminologie afspreken.

De actoren waar ik dacht waren:

Denk dat het chill is om morgen misschien samen te zitten (genen die aan klasse diagram gezeten hebben en use case diagram) om hier afspraken over te maken om zo verwarring te voorkomen.

TheAaltje23 commented 3 months ago

Ik heb wat kleine toevoegingen gedaan aan de UML om het wat completer te maken en die we wellicht morgen kunnen bespreken. De aanpassingen staan vermeld met twee sterretjes (**).

TheAaltje23 commented 2 months ago

adhv de stakeholder meeting (28-03) zullen we wat dingen moeten aanpassen aan de UML. Ik heb snel wat bulletpoints genoteerd, staat in de drive onder Modellen.