nehav39 / intelehealth-openmrs-module

Mozilla Public License 2.0
1 stars 1 forks source link

Clicking on the Patient Name in the Patient header links to the clinician facing patient dashboard #40

Closed nehav39 closed 6 years ago

nehav39 commented 7 years ago

"Clinician facing patient dashboard" == Page from the Ref App.

nehav39 commented 6 years ago

https://github.com/openmrs/openmrs-module-coreapps/blob/master/omod/src/main/webapp/fragments/patientHeader.gsp

nehav39 commented 6 years ago

You need to overload the following variables in to redirect the url to the Intelehealth patient dashboard - contextPath and config.dashboardUrl

See jq(".demographics .name").click(function () { emr.navigateTo({ url: "${ ui.urlBind("/" + contextPath + config.dashboardUrl, [ patientId: patient.patient.id ] ) }" }); })

https://github.com/openmrs/openmrs-module-coreapps/blob/master/omod/src/main/webapp/fragments/patientHeader.gsp

AniketIntelhealth commented 6 years ago

@neha @amalafrozalam this is regarding issue no #40. i did some research and came to the conclusion that while overriding the default core apps url we can solve this issue. but the problem is when overrided with intelehealth dashboard url with the default value its not working. can any 1 help me here out? also there is something called model.add attribute property in intelehealth dashboardurl.gsp page which is responsible for adding core apps default url.

AniketIntelhealth commented 6 years ago

screenshot from 2017-12-05 12-13-07

AniketIntelhealth commented 6 years ago

@amalafrozalam in intelehealthPatientDashboard.gsp there is fragment called ${ ui.includeFragment("coreapps", "patientHeader", [ patient: patient.patient, activeVisit: activeVisit, appContextModel: appContextModel ]) }

which is responsible for routing to core app url. if we can replace this url with "openmrs/intelehealth/intelehealthPatientDashboard.page?patientId=" + v.visit.patient.uuid, v.visit.patient) }" this fragment. the issue can be solved . (according to me)

nehav39 commented 6 years ago

@prithi69 can you test this?

AniketIntelhealth commented 6 years ago

http://localhost:8081/openmrs/admin/maintenance/globalProps.form

AniketIntelhealth commented 6 years ago

https://talk.openmrs.org/t/privilege-to-edit-patient-demographic-information/11520 @neha can you go through the link and help me out what "darius" wants to say

nehav39 commented 6 years ago

So it looks like someone else also wanted this feature. But per Darius's comment, this functionality does not exist.

prithi69 commented 6 years ago

patient header links to clinician dashboard, not our inteleealth dashboard.tested it but its not working..?? have you tested it in your local server..? @AniketIntelhealth

AniketIntelhealth commented 6 years ago

so what can be done for sure is to add your own extension through your custom module, the same way the Registration App module does it And then, to make sure that your custom extension is being picked up, you could disable the “default” one (=the one defined by the Reg. App module) through your custom module’s activator. With something like that:

AppFrameworkService service = Context.getService(AppFrameworkService.class); service.disableExtension("org.openmrs.module.registrationapp.editPatientDemographics");

--Dimitri's words on changing the edit button on patient Dashboard

AniketIntelhealth commented 6 years ago

screenshot from 2017-12-11 17-50-33 @nehagoel2 @amalafrozalam i made a patientHeader.gsp file with a controller called PatientHeaderFragmentController.java but when i replace the coreapp fragment with created fragment it gives the above error any help?

prithi69 commented 6 years ago

Patient Header is been disabled, but edit button still persist. So we may close the issue.