smart-on-fhir / health-cards

Health Cards Framework: implementation guide and supporting material
Other
261 stars 84 forks source link

Response code 404 (Not Found) #234

Closed yarodevuci closed 2 years ago

yarodevuci commented 2 years ago

TTPError: Response code 404 (Not Found) at Request. (health-cards-main/generate-examples/node_modules/got/dist/source/as-promise/index.js:118:42) at processTicksAndRejections (node:internal/process/task_queues:96:5)

Does anyone had same issue?

christianpaquin commented 2 years ago

Did you get this by running npm run generate-examples? Looks like the retrieved URLs need to be updated. Thanks for the report.

christianpaquin commented 2 years ago

The files Scenario1Bundle.json and Scenario2Bundle.json are no longer in https://github.com/HL7/fhir-shc-vaccination-ig/tree/master/examples. Looks like they were moved by recent PRs, @masnick, but I'm not sure where.

jmandel commented 2 years ago

Indeed, https://github.com/HL7/fhir-shc-vaccination-ig/blob/master/examples/README.md and https://github.com/HL7/fhir-shc-vaccination-ig/blob/master/script/validate-examples still refer to these files (in addition to our dependency from the spec ;-)).

Would be good to bring them back.

masnick commented 2 years ago

Apologies, I did not realize this depended on the location of those examples.

We are in the process of moving the examples to inside the IG as part of ballot reconciliation. The new examples are at:

  1. "Scenario 1": https://github.com/HL7/fhir-shc-vaccination-ig/blob/master/input/examples/bundle-immunization-covid.json
  2. "Scenario 2": https://github.com/HL7/fhir-shc-vaccination-ig/blob/master/input/examples/bundle-immunization-monkeypox.json
  3. "Scenario 3": https://github.com/HL7/fhir-shc-vaccination-ig/blob/master/input/examples/bundle-lab-test-results-covid.json

We still have a GitHub Action to auto-generate full SMART Health Cards from these examples, but the output of the script has moved into its own orphan branch to avoid a messy history on master: https://github.com/HL7/fhir-shc-vaccination-ig/tree/master-examples

christianpaquin commented 2 years ago

Looks like the "scenario 1" file is different than what is in the current spec (3 doses vs 2). Our example 2 used a modification of the "scenario 1" file, but not one of these new examples. Looks like we have two choices: 1) archive our old example files as fixtures vs. external downloads and keep the spec as is, or 2) update the spec examples to map to the updated external Scenario 1 file.

jmandel commented 2 years ago

Agreed with the analysis! I think a good compromise might be:

  1. snapshot + internal copy the examples we care about, for our embedded examples
  2. link out to the IG for more detail/ other examples
yarodevuci commented 2 years ago

@jmandel @christianpaquin thanks for assistance. Is there a temp walkarounds ?

jmandel commented 2 years ago

@yarodevuci can you confirm what commands you're running and what you're trying to accomplish? (Will try to get a fix in place ASAP so no workaround is needed, in any case...)

jmandel commented 2 years ago

See fix in https://github.com/smart-on-fhir/health-cards/pull/236 (you can use this branch for now; we'll merge ASAP).

christianpaquin commented 2 years ago

Oh, didn't notice your PR @jmandel; I just created #237. I see you are using @masnick new example file with 3 doses for the spec example 00 and 01. We can certainly do this (better ongoing alignment). I've archived the old examples in the PR I just pushed.

yarodevuci commented 2 years ago

@jmandel running npm run generate-examples and trying to build examples.

jmandel commented 2 years ago

Thanks! You should be good to go now.

yarodevuci commented 2 years ago

yes the issue was in https://raw.githubusercontent.com/HL7/fhir-shc-vaccination-ig/master/examples/Scenario2Bundle.json that resulted a 404. Thanks for the fix.