sumangala28
(Jeyasumangala Rasanayagam)
July 13, 2018, 9:05pm
1
Hello Everyone,
This is my mid term presentation for Improved Built-In Reports module.
CC : @raff @k.joseph
Thanks,
Sumangala
7 Likes
Nice to see those improvements! Good work @sumangala28
2 Likes
sumangala28
(Jeyasumangala Rasanayagam)
July 17, 2018, 9:12am
4
raff
(Rafal Korytkowski)
July 17, 2018, 9:14am
5
Thanks @sumangala28 ! Good progress so far!
A few items to address:
Remove the location selector from Number of Admissions as it doesn’t make much sense. Same for Number of Discharges and Number of Transfers. Can we add a total as the last row and a pie chart under them?
The location selector should say “All” instead of “Select location”. Picking “All” should show data from all locations.
1 Like
sumangala28
(Jeyasumangala Rasanayagam)
July 17, 2018, 9:15am
6
Thanks @raff for the suggestions. I will add those to the reports as well.
sumangala28
(Jeyasumangala Rasanayagam)
July 23, 2018, 3:02pm
7
Hi @raff
I have addressed those changes in the project discussion thread,
Hi @raff
As you suggested here,
I have worked to add these following changes to the Number of Admissions Report,
Removed the location selector from the report
Added Total Count to the bottom of the table.
Please see the attach screen shot,
[image]
Please review this PR :
Thanks.
Thanks.