Page MenuHomePhabricator
Differential D15556

Clean up EditEngine implementation on Badges
ClosedPublic
Actions

Authored by chad on Apr 1 2016, 12:31 AM.
Tags
None
Referenced Files
F14813076: D15556.id37508.diff
Mon, Jan 27, 9:54 AM
Unknown Object (File)
Fri, Jan 24, 12:37 PM
Unknown Object (File)
Fri, Jan 24, 12:37 PM
Unknown Object (File)
Fri, Jan 24, 12:36 PM
Unknown Object (File)
Fri, Jan 24, 12:36 PM
Unknown Object (File)
Tue, Jan 21, 9:17 AM
Unknown Object (File)
Sat, Jan 18, 1:23 AM
Unknown Object (File)
Sun, Jan 5, 11:32 AM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
lpriestley
Maniphest Tasks
T10672: Complete EditEngine implementation in Badges
Commits
rP154234dd1f04: Clean up EditEngine implementation on Badges
Summary

Fixes T10672. Cleaning this up myself since I was responsible for the implementation.

Test Plan

Leave a comment, Edit a badge, create a badge.

Diff Detail

Repository
rP Phabricator
Branch
badges-edit-engine-2 (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 11372
Build 14159: Run Core Tests
Build 14158: arc lint + arc unit

Event Timeline

chad updated this revision to Diff 37507.Apr 1 2016, 12:31 AM
chad retitled this revision from to Clean up EditEngine implementation on Badges.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added reviewers: epriestley, lpriestley.
chad added a task: T10672: Complete EditEngine implementation in Badges.
Herald added a subscriber: Korvin. · View Herald TranscriptApr 1 2016, 12:31 AM
epriestley accepted this revision.Apr 1 2016, 12:32 AM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Apr 1 2016, 12:32 AM
Closed by commit rP154234dd1f04: Clean up EditEngine implementation on Badges (authored by chad, committed by chad). · Explain WhyApr 1 2016, 12:33 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
badges/
application/
4 lines
controller/
13 lines
26 lines
editor/
17 lines

Diff 37507

View Options

src/applications/badges/application/PhabricatorBadgesApplication.php

Loading...
View Options

src/applications/badges/controller/PhabricatorBadgesListController.php

Loading...
View Options

src/applications/badges/controller/PhabricatorBadgesViewController.php

Loading...
View Options

src/applications/badges/editor/PhabricatorBadgesEditEngine.php

Loading...
Phabricator · Built by Phacility