Skip to content

Conversation

@flackjapz
Copy link
Contributor

@flackjapz flackjapz commented Oct 5, 2025

@flackjapz flackjapz requested review from a team as code owners October 5, 2025 12:45
@flackjapz flackjapz requested review from csande and removed request for a team October 5, 2025 12:45
@aws-amplify-us-west-2
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-850.d298pum72820gn.amplifyapp.com

milosdjakovic-vicert and others added 3 commits October 21, 2025 19:39
- Add ServiceRequest to Provenance resource documentation
- Add _revinclude search parameter to ServiceRequest endpoint with Provenance:target option
Copy link
Collaborator

Choose a reason for hiding this comment

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

Revert changes to this file.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Comment on lines 34 to 40
- value: active
- value: draft
- value: on-hold
- value: revoked
- value: completed
- value: entered-in-error
- value: unknown
Copy link
Collaborator

Choose a reason for hiding this comment

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

Update this list to reflect which statuses we are supporting.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

attributes:
- name: reference
type: string
description: The reference string of the subject in the format of `"Patient/<id>"`.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Use a realistic patient ID example here instead of <id>.

attributes:
- name: reference
type: string
description: The reference string of the requester in the format of `"Practitioner/<id>"`.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Use realistic ID.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

type: string
description: Type the reference refers to (e.g. "Practitioner").
- name: reasonReference
description: Reason for the request. Optional (0...N). References Conditions from Canvas.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
description: Reason for the request. Optional (0...N). References Conditions from Canvas.
description: Reason for the request. References Conditions from Canvas.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

attributes:
- name: reference
type: string
description: The reference string of the reason in the format of `"Condition/<id>"`.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Use realistic ID.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Comment on lines 149 to 155
- value: active
- value: draft
- value: on-hold
- value: revoked
- value: completed
- value: entered-in-error
- value: unknown
Copy link
Collaborator

Choose a reason for hiding this comment

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

Update this list.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Comment on lines 171 to 175
- name: _revinclude
description: Include Provenance resources that reference this ServiceRequest.
type: string
search_options:
- value: "Provenance:target"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Look at other documentation to see if we include this. If we do we can leave it here, but if we typically don't, let's remove it for consistency.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

- value: completed
- value: entered-in-error
- value: unknown
- name: intent
Copy link
Collaborator

Choose a reason for hiding this comment

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

Remove

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants