Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

changes for cds-hooks and cds-hooks-library (#1) #1108

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

rizzolol
Copy link

@rizzolol rizzolol commented Nov 4, 2024

See meeting minutes for "CDS Hooks Errata Review" at CDS 2024-10-30 Meeting Agenda. Per the minutes, our intention is to change the name of library specification in Jira from:

CDS Hooks patient-view (FHIR) [FHIR-cds-hooks-patient-view] to
CDS Hooks Library (FHIR) [FHIR-cds-hooks-library]

This commit does not do this (need administrative approval), just adds cds-hooks-library and necessary changes.

Copy link
Collaborator

@lmckenzi lmckenzi left a comment

Choose a reason for hiding this comment

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

You can't have the same base URL for two different specs.

Copy link
Collaborator

@lmckenzi lmckenzi left a comment

Choose a reason for hiding this comment

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

""https://www.hl7.org/fhir/uv/cds-hooks-library/" should be ""https://hl7.org/fhir/uv/cds-hooks-library". To confirm, "cds-hooks-library" has been approved by the FMG as the code for this IG in your IG proposal?

@rizzolol
Copy link
Author

rizzolol commented Nov 8, 2024

@lmckenzi after further discussion with members from the CDS workgroup, updating the cds-hooks-library link to:

http://cds-hooks.hl7.org/hooks

@lmckenzi
Copy link
Collaborator

lmckenzi commented Nov 9, 2024

Has the FMG approved the posting location?

@rizzolol
Copy link
Author

@lmckenzi we do not have approval yet, I will update this request after we have received it

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.

2 participants