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

Consent updated examples #986

Merged
merged 1 commit into from
Sep 22, 2023
Merged

Consent updated examples #986

merged 1 commit into from
Sep 22, 2023

Conversation

RicardoCrespo-ora
Copy link
Contributor

@RicardoCrespo-ora RicardoCrespo-ora commented Sep 21, 2023

Description

Updated examples for Search by Id and Search by Actor with updated data which was previously showing outdated data

Before
image

After
image


Before
image

After
image


Consistency with formatting in description by adding backticks
Before
image

After
image

PR Checklist

  • Screenshot(s) of changes attached before changes merged.
  • Screenshot(s) of changes attached after changes merged and published.

Copy link
Contributor

@kckeim kckeim left a comment

Choose a reason for hiding this comment

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

Sorry you had trouble with JSON formatting behaving on this one, @RicardoCrespo-ora.

The changes you made look okay from my perspective.
I would add some backticks around the details in the "Description" column around the parameters in the consent.md file - we have them for a few of the examples, but not all of them.
e.g.
actor=RelatedPerson/1234
->
actor=RelatedPerson/1234


image


Beyond that, I like this

@kckeim
Copy link
Contributor

kckeim commented Sep 21, 2023

Thanks for fixing stuff, @RicardoCrespo-ora.

Approved from my side now

@RicardoCrespo-ora RicardoCrespo-ora merged commit c556fb9 into main Sep 22, 2023
1 check passed
@RicardoCrespo-ora RicardoCrespo-ora deleted the consent-examples-update branch September 22, 2023 14:05
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