practable / bookjs

Javascript booking client for timdrysdale/relay
GNU Affero General Public License v3.0
1 stars 1 forks source link

Don't remember groups or sessions #20

Open timdrysdale opened 1 year ago

timdrysdale commented 1 year ago

Problem: We always show the everyone group to all users, but this confuses users in our labs.

Evidence: Users are reporting finding it confusing to see lots of everyone experiments when they are in a lab and being told to use a specific bit of kit (do they use the prebooked one or book another has been asked of our demonstrators in the labs).

Cause: Defaulting to always show the everyone group. This is compounded by remembering groups. These both seemed like a nice idea but the convenience of being able to access everything turns out to be less beneficial than being able to focus users in on just the right kit for their session.

Proposed fix: only show groups, sessions that are in the query params. Do not show everyone all the time.

Touches: the startup machine, the login machine. We should probably still add all the relevant groups, so that bookings can be made, but only fetch the ones that were specified in the URL query params. We'll need to filter the list of groups we request details for, rather than just requesting all.

Benefit: we reduce load on the server from supplying data on groups that are not required at the time.