Skip to content

Ballot notes pass for Resources - #4284

Merged
grahamegrieve merged 3 commits into
masterfrom
ballot-notes
Jul 17, 2026
Merged

Ballot notes pass for Resources#4284
grahamegrieve merged 3 commits into
masterfrom
ballot-notes

Conversation

@GinoCanessa

Copy link
Copy Markdown
Contributor

HL7 FHIR Pull Request

Note: No pull requests will be accepted against ./source unless logged in the HL7 Jira issue tracker.

If you made changes to any files within ./source please indicate the Jira tracker number this pull request is associated with:

Description

Pass of ballot-notes for resources. Note that I have NOT reviewed pages for the same.
Build is same as master, with additional file "*-introduction.xml": div/div[0] must start with an h2 for many ballot comments.

@GinoCanessa
GinoCanessa deployed to claude-triage July 17, 2026 02:15 — with GitHub Actions Active

@lmckenzi lmckenzi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

  • Inconsistent ballot-note markup. Most files use <div><blockquote class="ballot-note" id="bnN">…</blockquote></div>, but source/resource/resource-introduction.xml and source/domainresource/domainresource-introduction.xml use <div class="ballot-note" id="bn5">…</div> with no blockquote. Does the second one render correctly?
  • Empty id attribute. source/searchparameter/searchparameter-introduction.xml introduces <blockquote class="ballot-note" id="bn"> — the id value is empty; likely intended bn2 (or similar). Dunno if this is an issue
  • Editorial leakage. source/auditevent/auditevent-introduction.xml still contains the sentence "…includes unattributed mapping/follow-up cleanup that reviewers should tie to Jira before publication." That reads as an internal note; should it be visible to balloters?

@grahamegrieve
grahamegrieve merged commit 3118a7e into master Jul 17, 2026
5 checks passed
@grahamegrieve
grahamegrieve deleted the ballot-notes branch July 17, 2026 03:43
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.

3 participants