-
Notifications
You must be signed in to change notification settings - Fork 480
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 medical history [ Comorbidities ] #7852
Conversation
@AshrafMd-1 is attempting to deploy a commit to the Open Healthcare Network Team on Vercel. A member of the Team first needs to authorize it. |
✅ Deploy Preview for care-egov-staging ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Slightly changed the type interface medicalHistoryModel {
id?: number;
disease: string | number;
details: string;
extra_info?: object;
} the code is if (medData) {
const details = formData[`medical_history_${medData.id}`];
if (medData.id === 11) {
medical_history.push({
disease: medData.text,
details: details ? details : "",
extra_info: {
tb_status: formData["tb_status"] ?? "",
tb_duration: formData["tb_duration"] ?? "",
},
});
} else if (medData.id === 13) {
medical_history.push({
disease: medData.text,
details: details ? details : "",
extra_info: {
cancer_type: formData["cancer_type"] ?? "",
},
});
} else {
medical_history.push({
disease: medData.text,
details: details ? details : "",
});
}
} The payload it sends [
{
"disease": "Diabetes",
"details": ""
},
{
"disease": "OTHER",
"details": "Covid textdone 7"
},
{
"disease": "TB",
"details": "fvdvdf",
"extra_info": {
"tb_status": "Active",
"tb_duration": "4"
}
},
{
"disease": "Cancer",
"details": "rvrerveer",
"extra_info": {
"cancer_type": "Colorectal"
}
}
] |
@rithviknishad can you check this once |
👋 Hi, @AshrafMd-1, This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there. |
❌ Deploy Preview for care-net failed.
|
@rithviknishad can you check once |
@AshrafMd-1 fix the backend failure |
Can you check the backend once |
only the medical history option name and description is being fetched in the discharge summary , the additional details such as cancer type, TB status, etc.. all the additional details are missing |
@nihal467 fixed can you check once |
|
👋 Hi, @AshrafMd-1, This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there. |
Closing this PR as the entire medical history will be re-built after verifying proper standards and included in the health details API. |
Proposed Changes
@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
Merge Checklist