dhis2-devs team mailing list archive

Thread
Date

Pull request for /api/interpretations/chart/{chartId} to add new request param 'ou'

To: dhis2-devs@xxxxxxxxxxxxxxxxxxx
From: jhansi <jhansirk@xxxxxxxxxxxxxxxx>
Date: Tue, 27 Sep 2016 17:03:24 +0530
Cc: Cyril Bousquet <Cyril.BOUSQUET@xxxxxxxxxxxxx>, twmsf <twmsf@xxxxxxxxxxxxxxxx>, Poulami Biswas <poulami@xxxxxxxxxxxxxxxx>, Thomas RODRIGUEZ <thomas.rodriguez@xxxxxxxxxxxxx>, Megan Mcguire <Megan.McGuire@xxxxxxxxxxxxxxx>

Hello,

We are planning to use ‘/api/interpretations’ API in DHIS2.23 for adding comments to charts. 

We are able to create interpretation for charts and reports but, we are not able to add the organisationUnit to the interpretation for chart (/api/interpretation/chart/{chartId}) unlike the interpretation for report (/api/interpretation/reportTable/{reportId}) which accepts a request param ‘ou’. In order to provide this functionality to interpretation for chart API, we have added a non mandatory request param ‘ou’ which will accept the organisation unit similar to interpretation for report API. As mentioned, this is not a mandatory param, if it is not provided, it will fallback to the existing functionality.

We have sent a pull request with the above mentioned functionality to 2.23/2.24 and master branches. Please let us know if there is any issue.

Thanks,
Poulami/Jhansi

Follow ups

Re: Pull request for /api/interpretations/chart/{chartId} to add new request param 'ou'
From: Lars Helge Øverland, 2016-09-27