HomePhabricator

(stable) Fix one more "Reviewers" wire format issue

Description

(stable) Fix one more "Reviewers" wire format issue

Summary: Fixes T11010. This also needs to be inflated until we fix the whole client/server responsibility issue here.

Test Plan:

  • Created a revision while observing error log, no error.
  • Disabled "allow self accept", tried to make myself a reviewer, got rejected with an error message.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T11010

Differential Revision: https://secure.phabricator.com/D15966

Details

Provenance
epriestleyAuthored on May 23 2016, 5:42 PM
epriestleyPushed on May 25 2016, 6:01 PM
Reviewer
chad
Differential Revision
D15966: Fix one more "Reviewers" wire format issue
Parents
rP77d543ba4463: (stable) Promote 2016 Week 21
Branches
Unknown
Tags
Unknown
Tasks
T11010: Http502 when submitting or updating a diff
Build Status
Buildable 12331
Build 15595: Run Core Tests