Page MenuHomePhabricator

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

Authored by epriestley on Aug 29 2017, 7:44 PM.
Tags
None
Referenced Files
F13085715: D18493.diff
Wed, Apr 24, 11:54 PM
Unknown Object (File)
Fri, Apr 19, 6:46 PM
Unknown Object (File)
Thu, Apr 18, 12:29 PM
Unknown Object (File)
Tue, Apr 16, 11:21 PM
Unknown Object (File)
Tue, Apr 16, 5:35 PM
Unknown Object (File)
Tue, Apr 16, 8:37 AM
Unknown Object (File)
Sat, Apr 13, 2:09 AM
Unknown Object (File)
Thu, Apr 11, 8:10 AM
Subscribers
None

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
Branch
ptype
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 18207
Build 24483: Run Core Tests
Build 24482: arc lint + arc unit