Code review comment for lp://staging/~aaronp/rnr-server/modify-only

Revision history for this message
Michael Nelson (michael.nelson) wrote :

Hey Aaron! Just for the future, I don't think you need to resubmit.

I've just peeked through r 188..189, which look great... I think the JSONErrorMixin works well (certainly simplifies the code at the call-sites). Assuming the tests pass, I'll get this landed now. Thanks for your work!

review: Approve

« Back to merge proposal