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

Have each concept map have its own page #24

Merged
merged 2 commits into from
Sep 9, 2024
Merged

Conversation

mrinnetmaki
Copy link
Collaborator

The structure, and especially the automated rendering of the concept map, kind of requires this.

Also fix some errors and warnings from QA report.

And show some content on Terminology page under each heading. We could also hide the headings that have no content, but it may be better to have them, so headings have the same numbering when (if?) the other parts get some content...

The structure, and especially the automated rendering of the concept map, kind of requires this.
@@ -0,0 +1,98 @@
Instance: appointmentstatus-ajanvarauksentila
InstanceOf: ConceptMap
Title: "Appointment status to Ajanvarauksen tila"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Italics (Ajanvarauksen tila)?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Muuten juu, mutta Title-kentässä markdown ei tunnu toimivan. :(
Ehkä en vain osaa?

Title: "Appointment status to Ajanvarauksen tila"
Description: "Mapping between FHIR Appoinment status codes and the Finnish logical model [*Ajanvaraus - Ajanvarauksen tila*](https://koodistopalvelu.kanta.fi/codeserver/pages/classification-view-page.xhtml?classificationKey=1943) (oid `1.2.246.537.6.881`)."
Usage: #definition
* name = "FinnishAppointmentStatusCodeMap"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should we use "Fi" instead of "Finnish"?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Hmmm... Good point.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed in 2da0000

@mrinnetmaki mrinnetmaki merged commit ae47e3b into master Sep 9, 2024
1 check passed
@mrinnetmaki mrinnetmaki deleted the page-per-conceptmap branch September 9, 2024 12:30
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