FHIR Chat · docs / PR #435 Non substantive ballot changes · cds hooks/committers

Stream: cds hooks/committers

Topic: docs / PR #435 Non substantive ballot changes


view this post on Zulip Github Notifications (Nov 24 2018 at 21:29):

brynrhodes opened PR #435
from non-substantive-ballot-changes to master

Addresses the following ballot comment issues:

#238: Clarified how prefetch templates are processed
#239: Clarified prefetch data conformance language
#242: Added definition for prefetch key
#244: Updated conformance language usage in prefetch restrictions
#246: Clarified prefetch limitations are on prefetch templates
#257: Added clarification on responses and notes about multiple services from the same hook
#262: Added examples for each action type
#264: Clarified documentation for actions
#274: Clarified hook context table is an example
#281: Clarified prefetch response resources may be a searchset bundle
#282: Provided rationale for the use of __null__ in the prefetch response
#287: Clarified allowable data types for context fields
#289: Changed prefetch token status change to Major
#298: Added examples to hook/prefetch discussion
#300: Added an overview section discussing CDS Hooks components
#313: Clarified definitions of prefetch template, prefetch token, and prefetch key
#314: Clarified prefetch behavior for multiple services attached to the same hook
#316: Added rationale for not allowing next page links in prefetch data
#344: Clarified distinction between context and prefetch
#343: Updated prefetch example to use full key names
#349: Corrected capitalization of conformance language in prefetch description

view this post on Zulip Github Notifications (Nov 24 2018 at 21:29):

brynrhodes review_requested PR #435

view this post on Zulip Github Notifications (Nov 24 2018 at 21:29):

brynrhodes review_requested PR #435

view this post on Zulip Github Notifications (Nov 24 2018 at 21:29):

brynrhodes review_requested PR #435

view this post on Zulip Github Notifications (Nov 30 2018 at 23:42):

brynrhodes synchronized PR #435

view this post on Zulip Github Notifications (Nov 30 2018 at 23:43):

brynrhodes commented on PR #435

@brettmarquard , I just committed some additional changes to the prefetch templates section that I hope address your concerns. Please take a look and let me know.

view this post on Zulip Github Notifications (Dec 03 2018 at 15:41):

kpshek synchronized PR #435

view this post on Zulip Github Notifications (Dec 04 2018 at 15:55):

brynrhodes synchronized PR #435

view this post on Zulip Github Notifications (Dec 05 2018 at 16:04):

brynrhodes synchronized PR #435

view this post on Zulip Github Notifications (Dec 05 2018 at 16:16):

brynrhodes synchronized PR #435

view this post on Zulip Github Notifications (Dec 05 2018 at 16:17):

brettmarquard edited PR #435
from non-substantive-ballot-changes to master

Addresses the following ballot comment issues:

Fixes #238: Clarified how prefetch templates are processed
Fixes #239: Clarified prefetch data conformance language
Fixes #242: Added definition for prefetch key
Fixes #244: Updated conformance language usage in prefetch restrictions
Fixes #246: Clarified prefetch limitations are on prefetch templates
Fixes #257: Added clarification on responses and notes about multiple services from the same hook
Fixes #262: Added examples for each action type
Fixes #264: Clarified documentation for actions
Fixes #274: Clarified hook context table is an example
Fixes #281: Clarified prefetch response resources may be a searchset bundle
Fixes #282: Provided rationale for the use of __null__ in the prefetch response
Fixes #287: Clarified allowable data types for context fields
Fixes #289: Changed prefetch token status change to Major
Fixes #298: Added examples to hook/prefetch discussion
Fixes #300: Added an overview section discussing CDS Hooks components
Fixes #313: Clarified definitions of prefetch template, prefetch token, and prefetch key
Fixes #314: Clarified prefetch behavior for multiple services attached to the same hook
Fixes #316: Added rationale for not allowing next page links in prefetch data
Fixes #344: Clarified distinction between context and prefetch
Fixes #343: Updated prefetch example to use full key names
Fixes #349: Corrected capitalization of conformance language in prefetch description

view this post on Zulip Github Notifications (Dec 05 2018 at 16:29):

brynrhodes synchronized PR #435

view this post on Zulip Github Notifications (Dec 05 2018 at 16:34):

kpshek closed PR #435

view this post on Zulip Github Notifications (Dec 10 2018 at 07:50):

kpshek commented on PR #435

The changes in this PR also fixed #226


Last updated: Apr 12 2022 at 19:14 UTC