search

nikolai-b / leeds-bread-coop

0 stars 0 forks source link

Delivery report generating an error message #55

Closed LeedsBreadCo-op closed 9 years ago

LeedsBreadCo-op commented 9 years ago

When looking ahead using the calendar or the skip fwd a day buttons in delivery report going fwd more than two days generates an error

nikolai-b commented 9 years ago

can you send me a link?

LeedsBreadCo-op commented 9 years ago

http://leeds-bread-coop.herokuapp.com/admin/delivery_reports?date=2015-02-16

nikolai-b commented 9 years ago

ah, you made an order but you've delete the bread type. I think you probably shouldn't ever delete bread types, can you explain why you need to?

LeedsBreadCo-op commented 9 years ago

We had a duplicate after I created a new bread type called half sheet focaccia. I did this as I couldn't input 0.5 as an amount for a bread type and we sell focaccia in whole and half sheets. Then I deleted the 1 sheet of focaccia bread type. My bad

nikolai-b commented 9 years ago

I've moved all the orders with the old focaccia bread type to have double the old quantity and set them equal to the new focaccia half sheet type. I've also remove the ability to delete bread types, wholesale customers and collection points as discussed. In the future if you want to stop people selecting a bread type we can add an Deactivate check box which will simply stop it appearing as an option but they will never be remove.

Bread types (and collection points and everything else) can be edited but be aware that this will effect new and old orders (and subscribers). So if focaccia becomes 'herby Italian' then the focaccia orders that 164 placed last year will be renamed 'herby Italian'.