Supporting OpenMRS Reporting Module reports in the Bahmni Reports App (Quick fix, BAH-314)

[binduak (Himabindu Akkinepalli)]

Hi @mseaton, @darius

@mseaton

whether this actually works for all of the expected use cases, so if you can demonstrate this with tests and other examples then that will be helpful.

1. When we go to Report Administration page, We can see the below 3 report types.

These are all of ReportDefintion class type(subtypes are different though). I tried creating the ReportRequest for all the above 3 report types from POST MAN and they are working fine. We don’t get to know the report class type either from RequestBody or ResponseBody of the ReportRequest. One test case works for all the 3 report types. This is the reason why I couldn’t demonstrate much of the test cases here.

2. In the same Report Administration page, We can see the below 5 Report Definition types.

These are all of different class types(not of ReportDefinition class type). But we can create these report definitions(eg: Cohort Queries)from UI and add them as mappings to Period Indicator, Row-per-patient and Custom Report (I could see few of the combinations are not working, following this documentation helps). After adding them as mappings to any of the Period Indicator, Row-per-patient and Custom Report, I could successfully run them from POST MAN.

Now my questions are,

1. Do we have to run these 5 Report Definition types from POSTMAN ??
   I could see ReportRequest class allows only 2 types of Mapped classes. I.e ReportDefiniton and ChortDefinition.
2. What is exactly Mapped<CohortDefinition> baseCohort in ReportRequest class ?? Should it be only of “Cohort Query” type or any of different Cohort Queries( I could see Person Query and Encounter Query along with Cohort Query under Cohort Queries tab). I could see this is optional when sending the RequestBody for the POST call. How does sending the baseCohort value changes the ReportRequest output ??. Sorry I am unable to get one use case for this.

I could see there are lot of possibilities with the OpenMRS reporting Module. Every time I go and try something out, I am learning new stuff. How about once the above questions are clarified, we release a MVP(minimum viable product) in the first release (obiviously which will not have support for all the types of report types) and improving in the subsequent release by taking inputs from community. This is just my opinion. Others opinions are always welcome.

@darius Please let me know we can have a call sometime today or tomorrow to clarify the above questions.

cc/ @mksd @angshuonline @shruthipitta

Thanks !