coronasafe
Expanding Investigations Seed list
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
- [ ] Expand the seed data with tests highlighted here in the list : https://docs.google.com/spreadsheets/d/1hN13p-BDbb1Pkl3a8tcVdWFsFxncju2QPfeOEkKh51E/edit?usp=sharing
- [ ] Update reference values and names of some tests as highlighted in the same list
- [ ] Add tests to more groups, as highlighted in the same list
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.
I have a few doubt about the above changes and wanted to confirm before raising the PR.
include category_id:2mean that if it was4before , now updated category_id will be4,2previously 2what does this mean? and is the changes for this already made?not mentioned on Harsh mohan sheetat row 78, what does that mean ?
@aparnacoronasafe
@DraKen0009
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.
@aparnacoronasafe
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 pH – rows 230, 203, 209, 169, 220
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
@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
