- RA-1862: Reference Metadata should not install non built-in reports
I like this makes the reports loaded in the Reference Application explicit
This seems like the right thing to do, especially given this comment:
//setup built-in reports from #org.openmrs.module.referenceapplication.reports
My only suggestion is to name this something slightly more straight-forward: Reference Metadata should only install built-in reports.
Great catch @mksd , i liked the description on your ticket as in ReportManagerUtil.setupAllReports(ReportManager.class); may end up installing unwanted reports whose report manager happen to be available as a bean., it make alot of sense
Thanks all for your quick feedback and for getting this merged super quick Super helpful.