Stream: cds hooks/committers
Topic: docs
Github Notifications (Sep 21 2016 at 11:20):
kevin.shekleton pushed to branch master
- : Add deploy configuration with a port set option
- : Note the need for an upgraded Ruby
- : Add JavaScript examples
- : Merge pull request #464 from BaButtons/patch-1
- : Merge pull request #511 from Jameskmonger/patch-1
- : Update rouge languages link in README
- : Add PR template
- : Add issue template
- : Update readme
- : Remove CONTRIBUTING
[and 2 more commits]
Github Notifications (Sep 21 2016 at 14:26):
kevin.shekleton pushed to branch master
Github Notifications (Jan 14 2017 at 22:21):
kevin.shekleton@gmail.com pushed to branch link-type
- 9ff155a: Add new field type to links in order to distinguish between absolute and
Github Notifications (Jan 14 2017 at 23:46):
kevin.shekleton@gmail.com pushed to branch link-type
- 43f1034: Update CDS card example response with the new type field on links
Github Notifications (Jan 14 2017 at 23:53):
noreply@github.com pushed to branch master
- 9ff155a: Add new field type to links in order to distinguish between absolute and
- 43f1034: Update CDS card example response with the new type field on links
- f29eb72: Merge pull request #16 from cds-hooks/link-type
Github Notifications (Apr 14 2017 at 10:46):
kevin.shekleton@gmail.com pushed to branch master
- 7363828: Fix typo in medication-prescribe documentation to reference MedicationOrder
Github Notifications (Apr 14 2017 at 10:47):
kevin.shekleton@gmail.com pushed to branch gh-pages
- 7033cb3: publish: Fix typo in medication-prescribe documentation to reference MedicationOrder (DSTU2 final) instead of MedicationPrescription (DSTU2 May 2015 ballot). This is in line with the rest of the documentation where DSTU2 final is used throughout.
Github Notifications (Sep 09 2017 at 13:08):
Should be 'Authorization' (bad spelling)
Github Notifications (Sep 09 2017 at 13:12):
Thanks! Fixed in 3c510b6
Github Notifications (Nov 29 2017 at 17:48):
value will be sent to the SMART app
It might be good to say "as part of the OAuth2 access token response".
Github Notifications (Nov 29 2017 at 18:19):
Done - see PR #122
Github Notifications (Dec 22 2017 at 16:57):
JustinStauffer commented on b34534e
Can we model this more like a FHIR Extension (http://hl7.org/fhir/extensibility.html)? I don't think we have to get _quite as complicated_ as FHIR's extensibility model but using a key/value pair would be much more preferable than using dynamically named JSON properties.
For example, this is readable/writable using a standard JSON formatter/writer and does not require customizing out-of-the-box / off-the-shelf JSON libraries:
"services": [ { "extension": [{ "url": "http://domain/example-client-conformance", "value": "http://my.hooks.org/fhir/102/Conformance/patientview" }, { "url": "http://domain/example-client-another-extension", "value": "another-extension-value" }] }Notice that I simplified this a little from the FHIR Extension type in that I am just using one "value" property instead of "valueInt", "valueDecimal", "valueString", etc.
To conclude, I think it is important that we stop requiring customization to off-the-shelf JSON libraries when adding support for new features in CDS Hooks as it will make implementation of those features more difficult for everyone and therefore, slower to reach users (customers).
Github Notifications (Jan 30 2018 at 16:26):
brettmarquard commented on 888d3ad
should smart be uppercase (SMART) everywhere?
Github Notifications (Aug 01 2018 at 12:18):
I changed our DNS A records so that cds-hooks.org can be always served over HTTPS. This requires that you manually remove/re-add the CNAME here.
See https://help.github.com/articles/troubleshooting-custom-domains/#https-errors
Last updated: Apr 12 2022 at 19:14 UTC
