Fix up special casing for closing review requests.
Review Request #9150 — Created Aug. 25, 2017 and submitted — Latest diff uploaded
In the case of the close description field, the way that the review request
editor would callreviewRequest.closeinstead of setting a field on the model
itself was pretty hacky. The implementation ofsetDraftFieldwould look at
whether it was setting thecloseDescriptionand then special-case its
behavior.Now that we have JavaScript-side implementations for the built-in fields, we
don't need this anymore. I've moved the special casing into the
CloseDescriptionFieldView.This refactor will also make things easier down the road for new field types.
- Tested setting the close description with both regular and Markdown text.
- Ran js-tests.