New Growth Chart Module for Reference Application Released !

[mozzy (Moses Mutesasira)]

Alright @ddesimone , let me work on it

[cioan (Cosmin Ioan)]

Thanks @mozzy for implementing those charts. I have merged them in. Would be possible to add the growth chart module to the openmrs test server (http://qa-refapp.openmrs.org/)? THanks!

[mozzy (Moses Mutesasira)]

Yes , i was going to add to the reff app distribution , but we still have a few issues. We can probably add it to the http://modules-refapp.openmrs.org/ at the moment

cc @dkayiwa

[dkayiwa (Daniel Kayiwa)]

@mozzy

We can probably add it to the http://modules-refapp.openmrs.org/ at the moment

@mozzy you can add it from GitHub - dkayiwa/openmrs-distro-referenceapplication if you want it show up at modules-refapp

[mozzy (Moses Mutesasira)]

created PR here https://github.com/dkayiwa/openmrs-distro-referenceapplication/pull/1

[mozzy (Moses Mutesasira)]

i have deployed the growth charts module to the modules-refapp server..

@dkayiwa , you can give the credental testers will use to login into the server.

Note , For better testing , whowever will have to test will add the Head Circumfrence Concept , and set growthchart.headCircumferenceConceptUuid Global property to the right uuid.

[mozzy (Moses Mutesasira)]

@ddesimone

I would say RA-1663 (Growth Chart data points are difficult to see) is the next highest priority for these. Any chance you (or someone else) could take a look at this?

When this one is resolved we will release them in production at some of our sites.

@ddesimone , i have fixed that too. i have made the data points solid , and also incresead their size.
@cioan , you can merge that in and i re-deploy the updated omod to the modules-refApp server

[ball (Ellen Ball)]

@mozzy I looked at growth chart on Login. Uncertain about this popup. Does it only appear if the head circumference concept is not available and configured? The popup appears for a moment, so not sure if that’s helpful. (You need to modify the messages.properties with the English translation for growthchart.headCircumferenceG_P (?)).

Is it possible to only show the link+graph if the concept is available/configured?

[akanter (Andrew Kanter)]

Login/pswd?

[mozzy (Moses Mutesasira)]

@ball

Does it only appear if the head circumference concept is not available and configured

innitially , if the Head Circumfrence Concept is not present (NOTE : it was not then part of the default concepts that came out of the box with tge reff app) , the chart would throw a NPE. So i handled it , and wanted to notify the user that its not present and the growthchart.headCircumference Global property needs to be confiured.

That pop up doesnt appear when that concept is present and the that global property is rightly configured

[mozzy (Moses Mutesasira)]

@ball

growthchart.headCircumferenceG_P (?)

i was simply meaning growthchart.headCircumference Global property. I used a ui commons util component to flash that notification on the screen , on putting in alot of of text , it doesnt show properly.

What could be the most appropriate message for that notification ??

[ball (Ellen Ball)]

@mozzy

i was simply meaning growthchart.headCircumference Global property. I used a ui commons util component to flash that notification on the screen , on putting in alot of of text , it doesnt show properly.

What could be the most appropriate message for that notification ??

How’s this?

Add and configure head circumference concept. Growth Chart Module For Reference Application - Documentation - OpenMRS Wiki

[mozzy (Moses Mutesasira)]

great @ball

[mozzy (Moses Mutesasira)]

I have re-deployed the growth charts with the latest changes on the modules-reffapp-server

cc @ddesimone @ball @akanter

[ball (Ellen Ball)]

Great work @mozzy. This is a great addition and harvested from iSantePlus (Haiti).

Documentation: https://wiki.openmrs.org/display/docs/Growth+Chart+Module+For+Reference+Application

[mozzy (Moses Mutesasira)]

i have released version 2.1.0 , with the fixes i did.