FHIR Chat · docs / PR #153 Move patient and encounter into hook-speci... · cds hooks/committers

Stream: cds hooks/committers

Topic: docs / PR #153 Move patient and encounter into hook-speci...

Github Notifications (Jan 28 2018 at 16:20):

kpshek edited PR #153
from 119-context-update to master

This should address the outcome of the discussion in #119.

Github Notifications (Jan 28 2018 at 23:54):

mattberther synchronized PR #153

Github Notifications (Jan 29 2018 at 00:33):

bakerdb commented on PR #153

@kpshek It seems to me that "patient" should be OPTIONAL even when pulled within "context." Looks like #153 makes "encounter" optional, but not "patient.

Github Notifications (Jan 29 2018 at 00:53):

mattberther commented on PR #153

@bakerdb The optional/required element as described in this PR are intended to be hook-specific. This should not be interpreted to imply that context must always include a patientId or encounterId.

Can you think of a way we can make this clearer?

Github Notifications (Jan 29 2018 at 15:05):

kpshek synchronized PR #153

Github Notifications (Jan 29 2018 at 15:28):

kpshek commented on PR #153

@kpshek It seems to me that "patient" should be OPTIONAL even when pulled within "context." Looks like #153 makes "encounter" optional, but not "patient.

@bakerdb - For the three hooks we have defined today, patientId is REQUIRED for all of them. The reason is that for each of these hooks, you always have a patient in context. However, you may not have an encounter opened. This is actually have things have always worked previously with these hooks in CDS Hooks; we're just making that explicit here.

Github Notifications (Jan 29 2018 at 15:35):

kpshek synchronized PR #153

Github Notifications (Jan 29 2018 at 15:42):

kpshek closed PR #153

Last updated: Apr 12 2022 at 19:14 UTC