Code review comment for lp://qastaging/~thumper/launchpad/set-empty-review-type-as-none

Revision history for this message
Tim Penhey (thumper) wrote :

The failure to claim team reviews was primarily due to the mismatch between '' and None and the ways that the zope form machinery, the api and the email interface interact.

This branch normalizes the review_type at the model level to avoid this complication.

tests:
  TestBranchMergeProposalNominateReviewer

« Back to merge proposal