Page MenuHomePhabricator
Differential D19329

Modularize transactions for Almanac Device
ClosedPublic
Actions

Authored by epriestley on Apr 10 2018, 3:43 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 19, 6:04 PM
Unknown Object (File)
Fri, Apr 19, 2:52 AM
Unknown Object (File)
Thu, Apr 18, 4:59 AM
Unknown Object (File)
Thu, Apr 11, 8:57 AM
Unknown Object (File)
Wed, Apr 3, 9:35 AM
Unknown Object (File)
Sun, Mar 31, 11:44 AM
Unknown Object (File)
Sat, Mar 30, 4:42 PM
Unknown Object (File)
Sat, Mar 30, 4:42 PM
View All 85 Files
Subscribers
None

Details

Reviewers
amckinley
Maniphest Tasks
T12414: Implement Almanac edit endpoints in Conduit
T13120: Plans: 2018 Week 15 Bonus Content
Commits
rP71c77fcc3a15: Modularize transactions for Almanac Device
Summary

Depends on D19328. Ref T13120. Ref T12414.

Prior work has left us with just a NAME transaction here, which is straightforward to modularize.

Test Plan
  • Created and renamed devices.
  • Tried to set no name, a bad name, a duplicate name (got errors).
  • Tried to create/rename into a namespace I could not edit (got an error).
  • Grepped for AlmanacDeviceTransaction::.

Diff Detail

Repository
rP Phabricator
Branch
almanac14
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 20060
Build 27210: Run Core Tests
Build 27209: arc lint + arc unit

Revision Contents
Changeset List

Diff 46236

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/almanac/editor/AlmanacDeviceEditEngine.php

Loading...
View Options

src/applications/almanac/editor/AlmanacDeviceEditor.php

Loading...
View Options

src/applications/almanac/storage/AlmanacDeviceTransaction.php

Loading...
View Options

src/applications/almanac/xaction/AlmanacDeviceNameTransaction.php

Loading...
View Options

src/applications/almanac/xaction/AlmanacDeviceTransactionType.php

Loading...
Phabricator · Built by Phacility