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

Missing Category for Some Diagnosis #7938

Closed
gigincg opened this issue May 29, 2024 · 8 comments · Fixed by ohcnetwork/care#2207 or ohcnetwork/care#2248
Closed

Missing Category for Some Diagnosis #7938

gigincg opened this issue May 29, 2024 · 8 comments · Fixed by ohcnetwork/care#2207 or ohcnetwork/care#2248
Assignees
Labels
Backend bug Something isn't working

Comments

@gigincg
Copy link
Member

gigincg commented May 29, 2024

Describe the bug
Missing Category for Some Diagnosis

To Reproduce
In a consultation, Mark the Diagnosis as SD23 Forgetfulness disorder (TM1)

Expected behavior
All Diagnosis should be Categorized

Screenshots
image

@rithviknishad
Copy link
Member

rithviknishad commented May 29, 2024

@aparnacoronasafe

As per #6231, the chapter for these was marked to be removed. Since the categorisation under text uses this short name shown for showing beside the principal diagnosis, nothing is shown.

image image

Possible Solutions

  1. Fallback to using the full chapter name (eg: 26 Supplementary Chapter Traditional Medicine Conditions - Module I)
  2. Remove such diagnosis from our ICD 11 table.
  3. Disallow making such diagnosis as the principal diagnosis.
  4. Have a short name.
  5. Have a short name, but still keep it hidden in metabase.
  6. Fallback to using the root block.
image

@aparnacoronasafe
Copy link
Member

@rithviknishad please go ahead with (2) above. remove all ICD-11s in the chapter from the list itself

@gigincg
Copy link
Member Author

gigincg commented Jun 6, 2024

I still see Diagnosis without categories @rithviknishad

image

@gigincg gigincg reopened this Jun 6, 2024
@rithviknishad
Copy link
Member

@sainak Should we consider auto flushing redis on every re-deploy

@rithviknishad
Copy link
Member

@gigincg This is because redis was not flushed and loaded with new data. However, #2248 should fix it without needing that.

@gigincg
Copy link
Member Author

gigincg commented Jun 6, 2024

@rithviknishad We'll also need to migrate out data for the options we are blacklisting now

@rithviknishad
Copy link
Member

rithviknishad commented Jun 6, 2024

@gigincg we could either do:

  1. Set such record's as not principal diagnosis and set their verification status as "entered in error" (reversing this migration will not be possible).
  2. Soft delete such consultation diagnosis records entirely (reversing this migration is possible).

@gigincg
Copy link
Member Author

gigincg commented Jun 6, 2024

I think we could leave it as is, as long as the UI clearly shows such items are "Uncategorized" and don't allow creating further Principal Diagnosis with Uncategorized Options

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment