You're absolutely right. Thanks Jon... this is obviously unfinished. So
glad to have you check this over.


On Sun, May 26, 2013 at 4:55 PM, jonlemmon <notifications@github.com> wrote:

> In app/models/motion.rb:
>
> > @@ -7,7 +7,7 @@ class Motion < ActiveRecord::Base
> >    has_many :did_not_votes, :dependent => :destroy
> >    has_many :events, :as => :eventable, :dependent => :destroy
> >
> > -  validates_presence_of :name, :discussion, :author
> > +  validates_presence_of :name, :discussion, :author, :close_at_time_zone
>
> @robguthrie <https://github.com/robguthrie> @AaronThornton00<https://github.com/AaronThornton00>it'd be better if we're more strict about validating this with:
>
> validates_inclusion_of :time_zone, in:
> ActiveSupport::TimeZone.zones_map(&:name)
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/loomio/loomio/pull/731/files#r4393744>
> .
>