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

Support for order-sign Hook #152

Merged
merged 5 commits into from
Jun 9, 2024

Conversation

c-schuler
Copy link
Contributor

@c-schuler c-schuler commented Jun 7, 2024

Builds on PR #151

  • Added Rx Sign tab
  • Added actions to support workflow
  • Updated reducers to account for supply duration, intent and category (needed for opioid content)

Partially addresses #93

I could take steps in merging this functionality into the Rx View tab if that is preferable.

There is not a default order-sign service available in the initial CDS Service. To test use the following configuration:
FHIR Server: https://opioid-sandbox.cqframework.org/cdc/opioid-cds-r4/ehr/fhir
Patient ID: example-opioidcds
CDS Service Discovery: https://opioid-sandbox.cqframework.org/cdc/opioid-cds-r4/cds-service/cds-services

- Added Rx Sign tab
- Added actions to support workflow
- Updated reducers to account for supply duration, intent and category (needed for opioid content)
- Migrating componentWillReceiveProps to use getDerivedStateFromProps
- Removing unused state property
@jmandel
Copy link
Member

jmandel commented Jun 9, 2024

I'm torn between

  • At a high level "it feels right" to have a single flow that covers/triggers order view and sign at the appropriate points in the workflow (better simulating what a real clinical system would do). This argues for merging the view and sign UX into a single tab.
  • Pragmatically if a developer is working on a specific hook, it's nice to have a top level navigation point focused on that hook. This argues for keeping separate tabs.

... Anyway, bringing these improvements in place without undue delay is probably the right principle.

To test use the following configuration:
FHIR Server

We really should support deep links to bring all this configuration to bear. (I could swear we used to, but may have lost it in the rewrite.)

@jmandel jmandel merged commit 213c1b3 into cds-hooks:master Jun 9, 2024
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