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

HL7 2024-Jan: Resolve or Remove "TBD" Statements from Text #273

Closed
ToddCooper opened this issue Mar 6, 2024 · 2 comments · Fixed by #349
Closed

HL7 2024-Jan: Resolve or Remove "TBD" Statements from Text #273

ToddCooper opened this issue Mar 6, 2024 · 2 comments · Fixed by #349
Assignees
Labels
Comment Review Comment of some sort from somewhere sometime HL7 Ballot Issue is related to an HL7 Ballot comment resolution

Comments

@ToddCooper
Copy link
Collaborator

Section Number

Table A-1. Complete List of Existing Actors Utilized in this specification

Priority

  • High: Important issue where there is major issue to be resolved. Requires discussion and debate.

Issue

HL7 JAN2024 Ballot Comment: OTHER-2766 "Remove TBD content".

Remove TBD notes and associated text (e.g., if we don't know if Time Client needs to be included, remove it). File issues to track outstanding actions instead

Proposed Change

Review the two "TBD" statements in Table A-1 and either remove the TBD statement or the actor entry.

@ToddCooper ToddCooper added Comment Review Comment of some sort from somewhere sometime Comment NEW A submitted comment waiting to be reviewed and dispositioned HL7 Ballot Issue is related to an HL7 Ballot comment resolution labels Mar 6, 2024
@ToddCooper ToddCooper removed the Comment NEW A submitted comment waiting to be reviewed and dispositioned label Apr 12, 2024
@ToddCooper ToddCooper added this to the SDPi 1.4 review milestone Apr 12, 2024
@ToddCooper ToddCooper assigned ToddCooper and kenjfuchs and unassigned ToddCooper Apr 12, 2024
@JavierEspina
Copy link
Collaborator

JavierEspina commented Nov 7, 2024

To get this one rolling again, here's my proposal on what exactly to do Table A-1:

  1. Remove entire row for "Alert Aggregator" (+ remove 2nd sentence of the "Alert Reporter" description) - Rationale: I don't think it is used by the ACM Gateway or anywhere else in the current SDPi Profiles . Moreover in the TF-1 "Alert Aggregator" is not presented as an actor, but as a sub-component of the "Alert Reporter" Actor.
  2. Remove the TBD note below "Time Client" - Rationale: I think the answer to the question is yes, especially in the Time Client case, which is used in a grouping requirement in section 1.2.5

@ToddCooper ToddCooper assigned JavierEspina and unassigned kenjfuchs Dec 6, 2024
@JavierEspina
Copy link
Collaborator

2024-12-06 SDPi Friday call: no disagreement to make the two changes proposed in #273 (comment). Proceed to making those changes.

JavierEspina added a commit that referenced this issue Dec 6, 2024
@JavierEspina JavierEspina linked a pull request Dec 6, 2024 that will close this issue
2 tasks
@JavierEspina JavierEspina removed a link to a pull request Dec 6, 2024
2 tasks
@JavierEspina JavierEspina linked a pull request Dec 6, 2024 that will close this issue
2 tasks
JavierEspina added a commit that referenced this issue Dec 6, 2024
* Resolution of issue #273 as discussed

* Added CHANGELOG entry for issue #273
@github-project-automation github-project-automation bot moved this from New issues to Done in Gemini SDPi Releases Dec 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Comment Review Comment of some sort from somewhere sometime HL7 Ballot Issue is related to an HL7 Ballot comment resolution
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants