harvest-dev team mailing list archive

[Launchpad Bug Tracker <659355@xxxxxxxxxxxxxxxxxx>]

** Branch linked: lp:~dylanmccall/harvest/659355

-- 
OpportunityList.explanation field is not allowed to be blank
https://bugs.launchpad.net/bugs/659355
You received this bug notification because you are a member of harvest-
dev, which is the registrant for harvest.

Status in Harvest Service: New

Bug description:
If I open /admin and start editing an opportunity list, I can't save changes unless I set the Explanatory instructions field to something. That is not the correct behaviour; this should be an optional field. In addition, the field has null=True, which is not recommended for text fields.

Meanwhile, the experience field has two blank choices here. One generated by Django (with the label "-------"), and another by us (with a blank label). Removing blank=True fixes that, removing Django's own blank choice (which seems to map to an empty string, leading to misery and destruction). Unfortunately that also makes the admin interface show “Required Experience” as if it needs to be set to something, unlike a BooleanField for example, but I think we can deal with that.