print should be commented or removed ?

Why do we need this for milestone ? and not priorities ?

Should this be removed or commented ?

Code looks good, just one question and some cleanup recommendation. Otherwise LGTM :thumbsup:

Removed yes

No this is used to tell nosetests to display the entire diff instead of just a part of it

Good question, not quite sure :s

Looks like we always introduce an empty priority in the list of priorities (in the db) while we don't do that for milestones, so we do this here in the form

Ok, I think I answered all the question and removed that un-needed print statement :)

Thanks for the review!