Hi Mark, Ok. I'll keep the test disabled for now and we can enable it again once the fix is released on the API side.
Thanks, Paulo On Mon, Oct 10, 2016 at 8:28 AM Mark Polak <[email protected]> wrote: > Hey Paolo, > > We have made this action unavailable in the UI, as like Jason says, this > is generally not something you would want. > > I would say the test case is indeed correct. We should enforce the same > logic on the server. > > On Mon, Oct 10, 2016 at 6:32 AM, Jason Pickering < > [email protected]> wrote: > > Hi Paulo, > > I would expect that this operation should be disallowed by the server. The > reason for this is that once you have established that a given option set > has a type "INTEGER" , you should not be able to change this afterwards to > a different type. It seems the test is correct to me, but from the results > of the test, it seems it is possible to update. > > Maybe the developers can comment more? > > Regards, > Jason > > > On Sun, Oct 9, 2016 at 5:58 PM, Paulo Grácio <[email protected]> > wrote: > > Hi devs, > > should it be possible to update existent Option Set type from 'INTEGER' to > ‘TEXT'? > > Test Case: > https://github.com/dhis2/api-tests/blob/master/modules/data-admin/optionSet.js#L77 > Failing Job: https://travis-ci.org/dhis2/api-tests/jobs/166211140 > > BR, > Paulo Grácio > > _______________________________________________ > Mailing list: https://launchpad.net/~dhis2-devs > Post to : [email protected] > Unsubscribe : https://launchpad.net/~dhis2-devs > More help : https://help.launchpad.net/ListHelp > > > > > -- > Jason P. Pickering > email: [email protected] > tel:+46764147049 > > _______________________________________________ > Mailing list: https://launchpad.net/~dhis2-devs > Post to : [email protected] > Unsubscribe : https://launchpad.net/~dhis2-devs > More help : https://help.launchpad.net/ListHelp > > > > > -- > Regards, > > > Mark Polak > Software developer, DHIS 2 > University of Oslo > http://www.dhis2.org <https://www.dhis2.org/> > [email protected] >
_______________________________________________ Mailing list: https://launchpad.net/~dhis2-devs Post to : [email protected] Unsubscribe : https://launchpad.net/~dhis2-devs More help : https://help.launchpad.net/ListHelp
