Page MenuHomePhabricator
Differential D17850

Clean up some Phame transaction edit bugs
ClosedPublic
Actions

Authored by chad on May 8 2017, 9:54 PM.
Tags
None
Referenced Files
F15563563: D17850.id42934.diff
Wed, Apr 30, 9:31 AM
F15560112: D17850.id.diff
Tue, Apr 29, 1:05 PM
F15557695: D17850.diff
Mon, Apr 28, 10:48 PM
F15460516: D17850.id42931.diff
Mar 31 2025, 11:30 PM
F15443376: D17850.id42934.diff
Mar 27 2025, 4:06 AM
F15436477: D17850.id.diff
Mar 25 2025, 2:30 PM
F15435237: D17850.diff
Mar 25 2025, 7:28 AM
F15434495: D17850.id42934.diff
Mar 25 2025, 4:07 AM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T12685: Modular Transactions Migrations Errata
Commits
rPade380530f94: Clean up some Phame transaction edit bugs
Summary

Ref T12685. Sets required on required fields, cleans up mailtags on PhamePost

Test Plan

Create a new blog, post.

Diff Detail

Repository
rP Phabricator
Branch
phame-xaction-3 (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 16857
Build 22508: Run Core Tests
Build 22507: arc lint + arc unit

Event Timeline

chad created this revision.May 8 2017, 9:54 PM
Herald added a subscriber: Korvin. · View Herald TranscriptMay 8 2017, 9:54 PM
Harbormaster completed remote builds in B16857: Diff 42931.May 8 2017, 9:55 PM
epriestley accepted this revision.May 8 2017, 10:50 PM
This revision is now accepted and ready to land.May 8 2017, 10:50 PM
Closed by commit rPade380530f94: Clean up some Phame transaction edit bugs (authored by chad, committed by chad). · Explain WhyMay 8 2017, 10:55 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
phame/
editor/
1 line
1 line
storage/
6 lines

Diff 42931

View Options

src/applications/phame/editor/PhameBlogEditEngine.php

Loading...
View Options

src/applications/phame/editor/PhamePostEditEngine.php

Loading...
View Options

src/applications/phame/storage/PhamePostTransaction.php

Loading...
Phabricator · Built by Phacility