Edit/View mode for Active/Past visit : Diagnoses fragment about intelehealth-openmrs-module HOT 13 CLOSED

The diagnosis fragment does not show the entered diagnoses for a closed visit.

from intelehealth-openmrs-module.

For this patient, a diagnosis of "cough" and "fever of unknown origin" was added.

from intelehealth-openmrs-module.

in the patient overview screen "back to top" is not working.

from intelehealth-openmrs-module.

@hardikk08
Check src > webapp > resources > scripts > diagnoses > diagnoses.js

from intelehealth-openmrs-module.

Above comment is incorrect. It's definitely in that file, but somewhere further along.

The fragment seems to be saving the information correct, but it does not seem to be retrieving it correctly. Perhaps the URL call in one of the factories is incorrect. Specifically, when the page is loaded and previous/prior diagnoses are searched for.

from intelehealth-openmrs-module.

@amalafrozalam - No code was written to retrieve and display diagnoses. It's not a bug. This was just not completed.

from intelehealth-openmrs-module.

@nehagoel2 you tagged the wrong amal. Need to use my full username.

Okay, well that explains it. There is a retrieval of sorts going on, but I believe that issues #23 and #44 are all related.

from intelehealth-openmrs-module.

@nehagoel2 I wish you had pointed that out earlier @amalafrozalam I can see it being posted in the database. So basically the code to retrieve it is missing. Would have been easy if we knew the logic flow of the entire module hence it is taking time to understand.

from intelehealth-openmrs-module.

Look at closed issue #26

from intelehealth-openmrs-module.

https://github.com/openmrs/openmrs-module-coreapps/tree/master/omod/src/main/webapp/fragments/diagnosis

from intelehealth-openmrs-module.

https://github.com/openmrs/openmrs-module-coreapps/blob/master/omod/src/main/java/org/openmrs/module/coreapps/fragment/controller/DiagnosesFragmentController.java

from intelehealth-openmrs-module.

@hardikk08 - great job so far on the fragment. Just a last push:

1. For a closed visit, the text entry box for entering diagnosis should be hidden so that no new diagnoses can be entered
2. The diagnoses being seen in the Previous diagnoses are not marked as Primary, Secondary, Confirmed or Unconfirmed. These should also be seen
3. Previous diagnosis heading is in Red - make it consistent in look and feel.

from intelehealth-openmrs-module.

For the 2nd point...
This is a bit difficult to achieve, saying it after trying a lot of stuff and this feature will take time. Reason, obs for diagnoses is being posted just after we enter a diagnoses name hence clicking on primary or confirmed has no effect on what is being posted in the database! Hence the checkbox implementation is more of a dummy right now. The way it is happening in core apps is very complex and will require a lot of modification in our app.

from intelehealth-openmrs-module.