-
Notifications
You must be signed in to change notification settings - Fork 624
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
Expanding Investigations Seed list #8510
Comments
I have a few doubt about the above changes and wanted to confirm before raising the PR.
|
1- yes. You are correct. Keep both classes 2- replace group 2 with the new group mentioned 3- it's just for internal reference. You can ignore that. Sorry, I should have cleaned it up. |
We currently have multiple investigation names that have the same values, which Ideally should have being unique. Here are a few I found: investigation name Additionally, at row 154, we need a numerical value in the "min" column. cc : @gigincg |
@DraKen0009 The issue of pH coming up multiple times is resolved. It is pH of various fluied. I have corrected it my providing the fluid type in "()" against pH. The missing min value in row 154 will be resolved shortly. getting the data |
@DraKen0009 the info in row 154 is edited to all required data. |
@aparnacoronasafe do we need to update the list right away or can we wait for the next release feature to come up, that should solve this issue permanently. cc @bodhish |
Let's wait for the Snowmed CT integration |
Is your feature request related to a problem? Please describe.
CARE investigations list is very limited. Many important tests are missing in the list
Describe the solution you'd like
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
The text was updated successfully, but these errors were encountered: