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

Migrate core publication to IG publisher #640

Open
wants to merge 24 commits into
base: master
Choose a base branch
from

Conversation

vadi2
Copy link
Collaborator

@vadi2 vadi2 commented Jun 24, 2023

This PR aims for a 1:1 translation from mkdocs to IG Publisher publication format. No parts of the specification will be changed, i.e. inputs and outputs will be exactly the same (typos and all).

Preview builds are available here.

Work done so far:

  • transferred index.md as-is
  • moved sections of spec out to separate pages
  • switched to hl7 template
  • updated sushi-config with the right properties
  • migrated mkdocs-specific syntax, such as !!! note
  • applied {:.grid} to all markdown tables to better reflect existing spec rendering in IG publisher view
  • escaped {{ with {% raw %}{{{% endraw %} so it renders properly in markdown
  • fixed italisation rendering by changing syntax from *_ to just _
  • migrated OAuth 2.0 links
  • removed <nobr> elements, they were breaking rendering and don't seem necessary in IG publisher
  • fixed broken links due to page organisation

Out of scope for this PR:

  • migrating each of the hooks to their own IGs

FYI:

  • this IG shares the "no resources defined" warning in QA with FSH, as neither IGs have any resource profiles

@vadi2 vadi2 marked this pull request as ready for review August 2, 2023 12:22
@vadi2 vadi2 changed the title Migrate publication to IG publisher Migrate core publication to IG publisher Aug 2, 2023
@vadi2
Copy link
Collaborator Author

vadi2 commented Dec 13, 2023

@isaacvetter @brynrhodes anything on this?

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