Right now if you want to disable any of our chores you can set the period to be <= 0. Sometimes, if you do this you get a nice message:
And sometimes you get an opaque message:
This is because sometimes we just blindly submit to ChoreService which submits to a java ScheduledExecutorService and then catches the IllegalArgumentException.
We should remove the one-offs and make it so ChoreService checks the period before accepting a submittal and produces a consistent "Foo is disabled" message.