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

J#46293 Removed orphaned/incomplete binding on Goal.target.detail #3156

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

Conversation

michelle-m-miller
Copy link
Contributor

@michelle-m-miller michelle-m-miller commented Aug 29, 2024

Description

J#46293 Removed orphaned/incomplete binding on Goal.target.detail (intentionally unbound)

@michelle-m-miller michelle-m-miller changed the title J#462936 Removed orphaned/incomplete binding on Goal.target.detail J#46293 Removed orphaned/incomplete binding on Goal.target.detail Aug 30, 2024
Copy link
Contributor

@lmckenzi lmckenzi left a comment

Choose a reason for hiding this comment

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

Should there not be at least a binding description?

@michelle-m-miller
Copy link
Contributor Author

@lmckenzi

Should there not be at least a binding description?

The vote was to remove the entire binding, including the description.

@lmckenzi
Copy link
Contributor

lmckenzi commented Sep 5, 2024

I think there's a methodology requirement that if you have a coded element you need to at least have a binding description if you don't have a binding - there needs to be some sort of guidance as to what the content should/might be. Can you start a discussion on Zulip?

@michelle-m-miller
Copy link
Contributor Author

I think there's a methodology requirement that if you have a coded element you need to at least have a binding description if you don't have a binding - there needs to be some sort of guidance as to what the content should/might be. Can you start a discussion on Zulip?

Patient Care is willing to update the element description to include "A CodeableConcept value could be Positive, Negative, Abnormal, Normal, Present, Absent, Yes, No."

@lmckenzi
Copy link
Contributor

That would work. Though we have codes that actually cover those concepts, so why not an example binding to point people to the existing codes if that's what they want? (Example bindings aren't intended to be comprehensive, just a set of codes that could theoretically be used.)

@michelle-m-miller
Copy link
Contributor Author

That would work. Though we have codes that actually cover those concepts, so why not an example binding to point people to the existing codes if that's what they want? (Example bindings aren't intended to be comprehensive, just a set of codes that could theoretically be used.)

When this was discussed in the past, the concern was that the values are so broad that any example value set is only a small fraction of possibilities and not very representative.

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