medplum / medplum / 30586138752

92%

push

github

fix(provider): Add support for /new in faxes and tasks (#10006)

* fix(provider): Add support for /new in faxes and tasks

Signed-off-by: David Yanez <me@davidyanez.com>

* refactor(provider): extract useNewInUrl and useControllableDisclosure hooks

Address review feedback on #10006: the /new-in-URL route glue and the
controlled/uncontrolled modal disclosure pattern were each copied across
three feature areas. Extract them into shared hooks and migrate FaxPage,
TasksPage, MessagesPage, FaxBoard, and TaskBoard.

Also dedupe the fax base path derivation in FaxPage and document why the
modals must close before firing their created callbacks.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Signed-off-by: David Yanez <me@davidyanez.com>

* clean up code

Signed-off-by: David Yanez <me@davidyanez.com>

* [autofix.ci] apply automated fixes

Signed-off-by: David Yanez <me@davidyanez.com>

* feat: log AuditEvents for Bot channel subscriptions (#9386)

* feat(server): always emit bot execution AuditEvent to logs

Bot execution AuditEvents (type 'execute') are now always emitted to the server logs
regardless of the bot's auditEventDestination setting. Writing to the database remains
optional and continues to be controlled by auditEventDestination (defaulting to resource
when unset).

This removes the separate subscription delivery (type 'transmit') AuditEvent that was
being created for bot-channel subscriptions. The existing bot execution AuditEvent
already captures the subscription as an entity, so a second event would duplicate
storage. The audit trail is preserved via log emission, which cannot be disabled.

Signed-off-by: Ian Plunkett <1857868+ianplunkett@users.noreply.github.com>

* docs: clarify bot vs rest-hook AuditEvent destination behavior

- subscription-extensions.md: add info callout that subscription-audit-event-destination
  applies to rest-hook subscriptions only, with a link to the bot docs for bot-channel
  subscriptions
- bots-in-produ... (continued)

22743 of 25823 branches covered (88.07%)

Branch coverage included in aggregate %.

40333 of 42717 relevant lines covered (94.42%)

12259.52 hits per line