search

praekelt / mama-ng-scheduler

A re-usable scheduler for MAMA Nigeria.
MIT License
0 stars 0 forks source link

Attempting to delete a schedule with messages returns a 404 #36

Closed hodgestar closed 9 years ago

hodgestar commented 9 years ago

This is very confusing. Instead we should either:

  1. Delete all messages associated with the schedule.
  2. Return a suitable response error explaining that the schedule still has messages.

I'm leaning towards the second option.

hodgestar commented 9 years ago

Ready for review (assuming Travis builds succeed).

imsickofmaps commented 9 years ago

:+1: looks right to my ungrailed eyes