Track previous consultation consultations

[rithviknishad]

Required Changes

• Schema Change: Add foreign key field: last_consultation (null if no previous consultation present)
• Custom Migration to populate the last_consultation field and add logic to fill this field for existing data.

[rithviknishad]

Adding this raises a concern, if a patient is readmitted to a different facility, the newer facility should not be able to access this information

[rithviknishad]

If the pre-readmission consultation's facility is not the same as the current consultation, the serializer should redact the consultation object.

[sainak]

@rithviknishad why don't we just return the external_id of the previous consultation and let the use navigate to the consultation? This would take care of the access control and would also avoid nested serialization

[rithviknishad]

Yup! That's much simpler and cleaner!

[rithviknishad]

(Possible since we now have the /consultation/id redirecting route in frontend

[Bhavik-ag]

Hey @rithviknishad I am working on this issue !! Please assign me.

[rithviknishad]

As discussed, let's add a field called: last_consultation instead of readmission_consultation.
This new field would be irrelevant to whether the current consultation is a readmission or not.
i.e.,

• If a patient had a second or Nth (n != 1) consultation filed even after 30 days from the previous consultation, we would still have last_admission with a value and readmission=False.

• If a patient had a second or Nth (n != 1) consultation filed within 30 days from the previous consultation discharge, we would still have last_admission with a value and readmission=True.

cc: @Bhavik-ag @sainak

[Omkar76]

Please assign me

[rithviknishad]

@Bhavik-ag unassigning due to no activity

[Bhavik-ag]

@rithviknishad Yup go ahead !!

[DraKen0009]

@Omkar76 If you aren't currently working on this issue , can I work on it?

[Omkar76]

Hi @DraKen0009, I'm done implementing the requirements. I was just verifying the migrations locally. I will be sending a PR for this one in few minutes.
Feel free to take unassigned issues from here https://github.com/coronasafe/care/issues?q=is%3Aissue+is%3Aopen+no%3Aassignee 😄