Skip to content
This repository has been archived by the owner on May 7, 2024. It is now read-only.

Updated DSTU2 Binary Resource #984

Merged
merged 4 commits into from
Sep 26, 2023
Merged

Updated DSTU2 Binary Resource #984

merged 4 commits into from
Sep 26, 2023

Conversation

kckeim
Copy link
Contributor

@kckeim kckeim commented Sep 20, 2023

General formatting updates.
Itemized list follows on Pull Request's Description
Verified good links throughout document, and have validated that example API calls behave as noted with examples

Description

  • Overview paragraph changed to indicate that the list of supported reports types is subject to change over time
  • Formatting for a lot of untagged "code" elements/parameters by adding backticks
  • Added links out to other R4 resources where referenced (e.g. DocumentReference and DiagnosticReport)

PR Checklist

  • Overview paragraph before
    image

  • Overview paragraph after
    image


  • Resources w/o links
    image

  • Resources w/ links
    image

General formatting updates
Itemized list follows on Pull Request's Description
Verified good links throughout document, and have validated that example API calls behave as noted with examples
Updated R4 links to point correctly back to DSTU2 resources
@@ -10,12 +10,13 @@ title: Binary | DSTU 2 API
## Overview

The Binary resource can contain any clinical content such as text, images, and PDFs. This resource is currently limited to Continuity of Care Documents (CCD), clinical documents, and diagnostic reports.
_The list of clinical documents and diagnostic reports is subject to change, as the list of clinical events that may qualify can be customized by clients_
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

period.


The consumer must populate the Accept header with either application/json+fhir or the format returned in the attachment.contentType of the referring resource. If the Accept header is application/json+fhir, a FHIR Binary resource is returned with the raw data populated in the content. Otherwise, the raw data will be returned (not contained within a FHIR resource). For more information see [the Binary Documentation](http://hl7.org/fhir/dstu2/binary.html#rest).
The consumer must populate the `Accept` header with either `application/json+fhir` or the format returned in the `attachment.contentType` of the referring resource. If the `Accept` header is `application/json+fhir`, a FHIR Binary resource is returned with the raw data populated in the content. Otherwise, the raw data will be returned (not contained within a FHIR resource). For more information see the HL7<sup>®</sup> FHIR<sup>®</sup> [Binary Documentation](http://hl7.org/fhir/dstu2/binary.html#rest).
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

<sup> is unnecessary


NOTE: To retrieve documents with application/pdf contentType, the client must have installed Cerner’s Clinical Reporting (XR) solution. For additional information about Cerner’s Clinical Reporting (XR) solution see [Clinical Reporting (XR) Reference Pages](https://wiki.ucern.com/display/reference/Clinical+Reporting+XR+Reference+Pages).
NOTE: To retrieve documents with `application/pdf` contentType, the client must have installed Cerner’s Clinical Reporting (XR) solution. For additional information about Cerner’s Clinical Reporting (XR) solution see [Clinical Reporting (XR) Reference Pages](https://wiki.ucern.com/display/reference/Clinical+Reporting+XR+Reference+Pages).
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure on the updated naming, but I know we've had some communication already. is something we should note for some point

@@ -31,14 +32,14 @@ List an individual Binary by its id:

_Implementation Notes_

* This is usually linked from DocumentReference or DiagnosticReport and should generally be accessed using the exact link given in the referring resource. Modifying the link has undefined consequences.
* This is usually linked from [DocumentReference](http://fhir.cerner.com/millennium/dstu2/infrastructure/document-reference/) or [DiagnosticReport](http://fhir.cerner.com/millennium/dstu2/diagnostic/diagnostic-report/) and should generally be accessed using the exact link given in the referring resource. Modifying the link has undefined consequences.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove "generally"

@@ -90,13 +91,13 @@ Generates the Continuity of Care Document (CCD) as a Binary for the supplied que

_Implementation Notes_

* This is usually linked from DocumentReference, and should generally be accessed using the exact link given in that resource. Modifying the link has undefined consequences.
* This is usually linked from [DocumentReference](http://fhir.cerner.com/millennium/dstu2/infrastructure/document-reference/), and should generally be accessed using the exact link given in that resource. Modifying the link has undefined consequences.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we should generally avoid vague language 😃

"Implementation Notes" sections had their language adjusted to be more direct by dropping the filler wording.
@gmodrogan gmodrogan merged commit 98fe6b4 into main Sep 26, 2023
1 check passed
@gmodrogan gmodrogan deleted the BinaryDSTU2-docs-migrate branch September 26, 2023 07:55
Surya-Rnsit pushed a commit that referenced this pull request Sep 28, 2023
 
* Review and corrections for ChargeItem (#976)
* typo fix for CapabilityStatement (#977)
* fixing URLs for scope and purpose in consent (#979)
* Updated DSTU2 care-plan (#980)
* Updated R4 CarePlan Resource (#981)
* Updated R4 CareTeam Resource (#982)
* Updated R4 CareTeam Resource (#983)
* Updated DSTU2 Binary Resource (#984)
* Review and Corrections for Device (#985)
* Consent updated examples (#986)
* Updated AllergyIntolerance R4 resource (#987)
* Custom encounter search (#989)
* Updated AllergyIntolerance DSTU2 resource (#990)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants