Page MenuHomePhabricator

Give Phriction documents modern string status constants instead of numeric constants
ClosedPublic

Authored by epriestley on Feb 15 2018, 8:56 PM.
Tags
None
Referenced Files
F15414856: D19100.id45766.diff
Thu, Mar 20, 2:49 AM
F15409871: D19100.diff
Wed, Mar 19, 5:44 AM
F15409450: D19100.diff
Wed, Mar 19, 4:32 AM
F15409121: D19100.diff
Wed, Mar 19, 3:20 AM
F15408900: D19100.diff
Wed, Mar 19, 2:07 AM
F15408724: D19100.diff
Wed, Mar 19, 12:54 AM
F15408532: D19100.diff
Tue, Mar 18, 11:42 PM
F15408348: D19100.diff
Tue, Mar 18, 10:30 PM
Subscribers
None

Details

Summary

Depends on D19099. Ref T13077. Updates Phriction documents to string constants to make API interactions cleaner and statuses more practical to extend.

This does not seem to require any transaction migrations because none of the Phriction transactions actually store status values: status is always a side effect of other edits.

Test Plan

Created, edited, deleted, moved documents. Saw appropriate UI cues. Browsed and filtered documents by status in the index.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable