Agreed!


2014-03-13 12:21 GMT-03:00 Adrián Pardini <notifications@github.com>:

> In models/App.js:
>
> > +        this.set('status', 'fixed');
> > +        this.set('end', moment().valueOf());
> > +    },
> > +    reopen: function() {
> > +        this.set('status', 'failing');
> > +        this.set('start', moment().valueOf());
> > +        this.set('end', 0);
> > +    },
> > +    defaults: {
> > +        code: -1,
> > +        description: "INVALID",
> > +        message: "INVALID",
> > +        status: "one-shot", // "one-shot" , "failing", "fixed"
> > +        reference: "",
> > +        sticky: false,
> > +        type: 'notification',
>
> On 13 March 2014 12:12, Juan Martin Runge <notifications@github.com>
> wrote: What other types are allowed? Could we have some kind of enumeration
> like with MostoCodes for status and type? Its useful to know what are the
> allowed values for this attrs.
> For now we only have 'error' and 'notification' in use and we set the type
> and status based on the contents of the App.MostoCodes map when we
> initialize the model. I think that instead of fiddling manually with them
> it is better to add another entry there. But adding a comment won't hurt.
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/inaes-tic/mbc-common/pull/62/files#r10569125>
> .
>