Page MenuHomePhabricator

Make legacy revision statuses from "differential.query" have type "string" again
ClosedPublic

Authored by epriestley on Aug 29 2017, 7:44 PM.

Details

Summary

Ref T2543. The type on these got changed by accident, it should be "string" (crazy nonsense, compatible) not "int" (sensible, not compatible).

(New API uses sensible strings like "accepted" only.)

Test Plan

Called differential.query from web UI, saw "2" and similar statuses.

Diff Detail

Repository
rP Phabricator
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.Aug 29 2017, 7:44 PM
jmeador accepted this revision.Aug 29 2017, 8:04 PM
This revision is now accepted and ready to land.Aug 29 2017, 8:04 PM
lvital accepted this revision.Aug 29 2017, 8:04 PM
This revision was automatically updated to reflect the committed changes.