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

2-5 Medication Treatment Plan document - XML Representation (Marcel Hanselmann) #297

Open
ig-feedback opened this issue Sep 23, 2024 · 9 comments · Fixed by #304
Open

2-5 Medication Treatment Plan document - XML Representation (Marcel Hanselmann) #297

ig-feedback opened this issue Sep 23, 2024 · 9 comments · Fixed by #304

Comments

@ig-feedback
Copy link
Collaborator

ch.fhir.ig.ch-emed#5.0.0-ci-build /Bundle-2-5-MedicationTreatmentPlan.xml.html

Example Instances: Since CH Core Medication has an optional GTIN code, it could be used in the Examples to identify the Medication.

Marcel Hanselmann

@qligier
Copy link
Member

qligier commented Sep 23, 2024

/Bundle-1-1-MedicationTreatmentPlan.html and /Bundle-2-3-MedicationTreatmentPlan.html are examples of MTPs with a GTIN.
2-5 is useful as an example without a GTIN, specifically.

@ziegm ziegm self-assigned this Sep 24, 2024
@ziegm ziegm added the STU 5 Ballot 2024 label Sep 24, 2024
@ziegm ziegm added this to the v5.0.0 milestone Sep 24, 2024
@ziegm
Copy link
Collaborator

ziegm commented Sep 27, 2024

The mentioned example https://fhir.ch/ig/ch-emed/Bundle-2-5-MedicationTreatmentPlan.xml.html does have a GTIN (urn:oid:2.51.1.1):

          <Medication>
            <id value="Norvasc"/>
            <meta>
              <profile
                       value="http://fhir.ch/ig/ch-emed/StructureDefinition/ch-emed-medication"/>
            </meta>
            <code>
              <coding>
                <system value="urn:oid:2.51.1.1"/>
                <code value="7680500440334"/>
                <display value="NORVASC Tabl 10 mg"/>
              </coding>
              <text value="NORVASC Tabl 10 mg"/>
            </code>

image

What we could do, to improve the readablity of the IG, we could add a more detailed description to the different examples here:
https://fhir.ch/ig/ch-emed/5.0.0-ballot/artifacts.html#4
Then it is more obvious where to find what.

@lpg-tech @pjolo
would you agree with that proposal?

@ziegm ziegm added the example label Sep 27, 2024
@pjolo
Copy link
Collaborator

pjolo commented Sep 27, 2024

@lpg-tech I am not sure if I have understood the issue correctly. Do you miss GTIN in the example or is it a statement that GTIN can be used?

@ziegm I agree with that proposal

@ganimedes
Copy link

I agree with the proposal, and the GTIN in the example would be helpful.

@ziegm
Copy link
Collaborator

ziegm commented Sep 27, 2024

I agree with the proposal, and the GTIN in the example would be helpful.

@lpg-tech The GTIN is already included. Or do you mean something different?

ziegm added a commit that referenced this issue Oct 2, 2024
@ziegm ziegm linked a pull request Oct 2, 2024 that will close this issue
ziegm added a commit that referenced this issue Oct 2, 2024
@ziegm ziegm closed this as completed in #304 Oct 2, 2024
@ziegm ziegm reopened this Oct 2, 2024
@ziegm
Copy link
Collaborator

ziegm commented Oct 2, 2024

@pjolo @lpg-tech please review the changes:

@ziegm ziegm assigned pjolo and unassigned ziegm Oct 2, 2024
@pjolo
Copy link
Collaborator

pjolo commented Oct 3, 2024

@pjolo @lpg-tech please review the changes:

@ziegm: From my side, the changes are good

@lpg-tech
Copy link

@ziegm @pjolo
Changes are perfect.
I still dont understand why the medication Reference is self-reflexive, and doesn't link the Medication 'Norvasc':
2-5 Medication Statement with Medication Norvasc-0-1

@ziegm
Copy link
Collaborator

ziegm commented Oct 15, 2024

because the medication is contained in the medicationstatement:

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants