We can't set it to empty string in the db w/o ensuring that won't screw up
lms.


On Fri, Jan 3, 2014 at 11:32 AM, Adam <notifications@github.com> wrote:

> In cms/templates/edit_subsection.html:
>
> > @@ -62,7 +62,7 @@ <h4 class="header">${_("Subsection Settings")}</h4>
> >      <div class="row gradable">
> >        <label>${_("Graded as:")}</label>
> >
> > -    <div class="gradable-status" data-initial-status="${subsection.format if subsection.format is not None else _('Not Graded')}">
> > +    <div class="gradable-status" data-initial-status="${subsection.format if subsection.format is not None else 'null'}">
>
> I actually prefer the empty string, because it preserves the intuition
> that grader_type = None means that grader_type is not at all supplied,
> where grader_type = '' would imply that grader_type is explicitly not
> graded. It's more elegant than the alternative I've posed here.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/edx/edx-platform/pull/2067/files#r8637836>
> .
>