medplum / medplum / 24256370874

92%

push

github

Scheduling Provider UI Fixups (#8915)

* Extract unchanging values from Calendar render

The localizer doesn't need to be re-instantiated each render cycle. Save
a few cycles by extracting it to a module level constant. It also is now
not referentially unique every render, which will help us eventually
prevent subtree renders.

The components object never changes, so I extracted it into a module
level constant for the same reason.

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

* Memoize some calendar props

These are being recomputed every render, which means that they are not
referentially stable and can cause expensive re-renders.

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

* Provider: Stop persisting 'transient' identifiers

When we get "virtual" slots back from the `$find` API, we're assigning
temporary IDs to them. This lets us assign them unique React keys when
showing them in lists, and identify these virtual slots when rendering
them on a calendar.

We never intended to actually persist these, but are submitting them as
part of $book operations. Oops! Let's keep these in our UI layer.

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

---------

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

18270 of 20816 branches covered (87.77%)

Branch coverage included in aggregate %.

33118 of 35126 relevant lines covered (94.28%)

13453.9 hits per line