search

clintonhealthaccess / chailmis-android

Android Tablet Application for Logistics Management in the DHIS2 Platform
Other
7 stars 6 forks source link

When Order module is selected and Routine Order type is selected, there should be no "select reason" drop down. Reason for order only exists for when Emergency Order type is selected #265

Open chai-lmis opened 8 years ago

garymabin commented 8 years ago

@chai-lmis, I find a unit test related to this issue, it shows that when Route Order type is selected, if user tries to change the start order date or end order date , the reason should shows, It that make sense ?

chai-lmis commented 8 years ago

start date and end date cannot be changed by the user this is fixed, start date is same as the 1st day of the current month & end date is the last day of the current month (not the next month as it currently shows)

garymabin commented 8 years ago

@chai-lmis, the start date and end date unmodifiable, I think some of the commodities are ordered bi-mounthly, some are monthly , which depends on "orderFrequecy" variable in the commodity data.

garymabin commented 8 years ago

Fixed 607a42

chai-lmis commented 8 years ago

Can only be tested when card 266 is fixed Routine Order Type should only show "reason for order" when the order quantity is not the same as the suggested quantity for order.