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

Actor & Interaction diagram #4

Open
standrd opened this issue Feb 12, 2021 · 1 comment
Open

Actor & Interaction diagram #4

standrd opened this issue Feb 12, 2021 · 1 comment

Comments

@standrd
Copy link

standrd commented Feb 12, 2021

Feedback op actor & interaction diagrammen

Actor diagram

  • Ik weet niet of mijn idee beter is, maar ik denk dat een router de render functies van een pagina aan spreekt, waarna deze de data verzameld uit de getData. In jouw geval spreekt de router nu de getData aan, van waar de render wordt aangeroepen. Hierdoor ben je voor het renderen erg afhankelijk van de API/data, waardoor je ook geen error state kunt renderen als de data niet goed binnenkomt (het kan natuurlijk wel, maar andersom zou dat mooier zijn).
  • De cleanData.js bevat nu de Javascript methods sort() & slice(), ik zou hier wat specifieker zijn welke functies jij daarmee gaat maken

Interaction diagram

  • Ziet er duidelijk uit, misschien kun je nog twee uitwerkingen doen: overview page & detail page
@HappyPantss
Copy link
Owner

Hey Stan, beetje late reactie, maar bedankt voor je feedback! Ik ga hier zo snel mogelijk aan werken.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants