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
F19908672: D15556.diff
Wed, Apr 1, 9:40 PM
F19863651: D15556.id37507.diff
Sat, Mar 14, 12:04 PM
F19809829: D15556.id37507.diff
Tue, Mar 3, 3:15 PM
F19749616: D15556.id.diff
Feb 15 2026, 8:33 PM
F19718673: D15556.id37508.diff
Feb 12 2026, 4:11 PM
F19712040: D15556.id37508.diff
Feb 11 2026, 8:04 PM
F18992023: D15556.diff
Nov 18 2025, 11:58 PM
F18942323: D15556.diff
Nov 11 2025, 11:57 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