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

Reduce calls to location on user management details page #15

Open
smessie opened this issue Nov 30, 2022 · 0 comments
Open

Reduce calls to location on user management details page #15

smessie opened this issue Nov 30, 2022 · 0 comments
Labels
diff: easy 0-1h performance prio: medium Operation is hindered, acceptable workaround is available

Comments

@smessie
Copy link
Member

smessie commented Nov 30, 2022

When going to a user on the user management details page, for every location reservation, a call to the locations/{id} is made, this can't be the way to do
either

  • include the location information in the original 'get location reservations' response
  • or, cache the location information so that for the each location, the call is only done once, e.g. map with locationId as key
@smessie smessie added diff: easy 0-1h prio: medium Operation is hindered, acceptable workaround is available performance labels Nov 30, 2022
@smessie smessie moved this to Todo in Studieplekken Nov 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
diff: easy 0-1h performance prio: medium Operation is hindered, acceptable workaround is available
Projects
Status: Todo
Development

No branches or pull requests

1 participant