medplum / medplum / 28688792096

92%

push

github

[Provider] Improve SchedulePage navigation + Schedule creation (#9727)

* Extract inner ScheduleDetails component from SchedulePage

This splits the responsibilities up a bit:
- Schedule page: selects a Schedule resource by actor, and loads it
- ScheduleDetails component: given a Schedule resource, loads the
  related Slot/Appointment resources and renders the calendar UI

This division will hopefully help keep things a little tidier; because
the inner component always knows it has a Schedule resource, it can
remove some checks. The outer component manages a much smaller set of
states, and provides a good place for us to hook other enhancements in
to the loading process as a follow up.

Signed-off-by: Noah Silas <noah@medplum.com>

* Improve SchedulePage navigation + creation

- When changing the viewed Practitioner, go into a loading state
  immediately instead of leaving the previous calendar on-screen. This
  reduces confusion on slow connections while the change is happening.

- When no Schedule is found for the selected practitioner, show
  an empty state that allows the viewer to create a Schedule
  resource for them. Previously, the last-viewed schedule would stay on
  screen, leading to confusing UI state.

- When first clicking in to the Scheduling screen from the sidebar, if
  we don't find an existing schedule for the viewer we show the empty
  state with a create button instead of automatically creating a new
  Schedule resource.

  - This is particularly nice if the viewer is a member of an
    operations team that should not have their own Schedule resource,
    but still would like to view the schedules of other Practitioners.

Implementation Notes:

- We consider the `id` parameter in the URL to be controlling. When
  navigating to a URL with this parameter, we fetch the schedule and
  display it.

- To select a different practitioner, you must first clear the search
  box, which navigates you to a page without an `id` parameter. Then ... (continued)

21688 of 24630 branches covered (88.06%)

Branch coverage included in aggregate %.

38617 of 40965 relevant lines covered (94.27%)

12548.11 hits per line