Cohort Module Enhancements Project Discussion

I think we all have OpenMRS setup on localhost and trying Travis would take substantial effort. So its fine to share the omod for feedback. I would test it and would create ticket for major issues

1 Like

Would him setting up CI/CD on Travis be hard? If not, it might be worth the effort, this way the community can see it…

Great. I’ve put up the omod already in the releases section

“Create tickets for major issues” Alright so I’ll have to work on the proposal timeline additions (essentially the weekly target) and these tickets simultaneously yes?

Hi @bholagabbar

Maimoona has covered all the points I would have recommended :slight_smile: Would like to see you progress as the pace your doing the work is good enough.

@maimoonak @sharonvarghese As the midterm evaluation approaches, I just wanted to discuss the current deliverables. The complete UI and all the pages will have been redone with bootstrap. I have not seen any tickets for resolving issues as you mentioned @maimoonak so I simply progressed ahead with the project.

As for the video presentation I was thinking that that I could visit every page in the module in a systematic way, showing how the module is used starting by creating a cohort, creating metadata etc.

As I put forward the issue 2 weeks back, I am unable to fetch the encounters for a cohort from the DB. I had posted the error log in the document we have maintained. Since it is a part of HFE, I think we can deal with this in the 2nd half.

As for the other issues, I still have to make a controller for search cohort and patient cohorts.

Besides this, as I mentioned, I am continuing with the UI of rest of the pages and it should be done by the end of this week. I will create the video, upload the same and fill in the midterm evaluation by Saturday, one day in advance just to be sure.

Do tell me if there are any issues from your side :slight_smile:

@bholagabbar, As mentioned in slack I was not able to find the omod. Could you please share the link of omod here. Moreover, Once you are done with video, share it with us and then we can provide our feedback and can fill the eveluation form as well.

Once we are done with this, lets spend sometime in trying setting up Cohort Module on a publically available platform (as Robby mentioned).

Oh OK I am no more active on Slack since I was instructed to keep messages public.

Here is the omod: https://github.com/OpenSRP/openmrs-module-cohort/releases

Sounds good otherwise.

Hi @bholagabbar I did update the document with the possible solution of the issue you were facing with fetching the encounters, please go through that . Will take a look into the omod file and let you know if any more changes can be done. Please finish your video by thursday as we can then suggest on what points you could improve. Do keep us updated with any more issues you face.

Thanks Sharon! Will go through it and let you know :slight_smile:

@sharonvarghese I"m referring to the document here but I do not see any updated changes. Could you see?

Hi @bholagabbar ,

I have created few issues in Sprint on OpenMRS here. Just assigning to you rightafter this.

A general problem I faced and did not create issues for is that clicking any button on dashboard opens old UI.

Moreover, please for next release update you version to 2.0.1. It is like major.minor.micro.

  • So for any change you made and it doesnot impact old installation version should be upgraded to 2.0.1/2/3/…
  • any change that adds databases changes or architectural changes should be 2.1/2/3/.x

Would keep adding issues there.

Ok great. Should I give priority to completion on old UI first ? And then deal with these issues?

Yeh its fine to reslove old UI issue first… But please note that most of these issues are critical and would impact the timelines of your Phase2 development.

Ok. Actually had the issues on sprint come up earlier, it would have been easier to deal with them then and there but now the project has moved forward so it’s an overhead to incorporate them.

I will continue with the UI part and side by side try to resolve the issues.

Hope that’s ok :slight_smile:

Hi @bholagabbar It has come under suggestions, just check it out or else I am posting the solution right here . Let me know if that issue is resolved

The reason the error has popped us the way its written as encounterDatetime in the java file while in the hbm file its encounterDateTime . Its the capitalization of the letters issue.

Hi Sharon and Maimoona,

I have put up a few comments on the tickets. Hope they can be addressed soon so I can proceed with corrections required in the UI. Thanks!

I added a screenshot on COH-7 and even a few queries. Would be able to proceed after that. Hope they can be addressed ASAP

@maimoonak , @sharonvarghese and myself are happy to announce the release of Cohort Module 2.0!.

This includes a completely revamped UI, A entensive search features and plenty of bug fixes among other things and is the final module release for GSoC 16. Among other things, I’ve also completed the final evaluation.

Cheers :slight_smile:

3 Likes

Hey @ssmusoke, saw your message asking for the location of the Cohort Module. As mentioned in Cohort Module Enhancements Project Discussion , the original location of the module is https://github.com/OpenSRP/openmrs-module-cohort but I have also taken the liberty of making a fork in the OpenMRS repository. Hence, you can access it at https://github.com/openmrs/openmrs-module-cohort :slight_smile: