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

#335 update quick start documentation #336

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

Conversation

alackerbauer
Copy link
Contributor

@alackerbauer alackerbauer commented May 25, 2023

updated the outdated information, old links and content to 2022 content

please note that:

the current response to the hook request does not contain any suggestion element
the CDS sandbox does return an empty list of cards.

Copy link
Collaborator

@c-schuler c-schuler left a comment

Choose a reason for hiding this comment

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

See comments for 2016 quick start PR (#337)

@c-schuler
Copy link
Collaborator

@alackerbauer This PR still needs the changes requested in the 2016 PR (#337)

@1anja1
Copy link
Collaborator

1anja1 commented Jun 17, 2023

All changes from the 2016 version have been applied, however, this error still occurs in the sandbox because the patient "patient-example-rec-10-patient-view-no-screenings" does not have a name element. Should we add one?
Screenshot 2023-06-17 at 18 23 54

@1anja1
Copy link
Collaborator

1anja1 commented Jun 26, 2023

Broken Links in 2016 Version

If the genonce script is run locally the links work without changing anything and this error does not occur. In the published version, however, it only works if a capital letter P is used in the filename: Patient-example-rec-10-patient-view-no-screenings.json
For the medication Request resources it works locally as well but for the published version the link has to be changed to MedicationRequest-example-rec-10-patient-view-no-screenings-context.json

These changes also work in the local version.
PR #346

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.

3 participants