Page MenuHomePhabricator
Differential D9841

Use PhabricatorEdgeType strings when rendering transactions
ClosedPublic
Actions

Authored by epriestley on Jul 5 2014, 10:48 PM.
Tags
None
Referenced Files
F14081398: D9841.diff
Fri, Nov 22, 7:37 PM
Unknown Object (File)
Thu, Nov 21, 7:48 AM
Unknown Object (File)
Sun, Nov 17, 1:03 PM
Unknown Object (File)
Wed, Nov 13, 4:24 PM
Unknown Object (File)
Wed, Nov 6, 7:15 PM
Unknown Object (File)
Wed, Oct 30, 3:29 AM
Unknown Object (File)
Oct 16 2024, 11:04 PM
Unknown Object (File)
Oct 16 2024, 11:04 PM
View All Files
Subscribers
epriestley

Details

Reviewers
joshuaspence
btrahan
chad
Maniphest Tasks
T5245: Migrate Maniphest Projects to use edge infrastructure
Commits
Restricted Diffusion Commit
rPb32313cc85d8: Use PhabricatorEdgeType strings when rendering transactions
Summary

Ref T5245. This hooks up the translation/rendering methods added previously.

These are messy, but now extractable/translatable.

Test Plan

Viewed edge transactions and stories, saw correct strings.

Diff Detail

Repository
rP Phabricator
Branch
projedge6
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 1543
Build 1543: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 23604.Jul 5 2014, 10:48 PM
epriestley retitled this revision from to Use PhabricatorEdgeType strings when rendering transactions.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: joshuaspence, chad, btrahan.
epriestley added a task: T5245: Migrate Maniphest Projects to use edge infrastructure.
Herald added a subscriber: epriestley. · View Herald TranscriptJul 5 2014, 10:48 PM
Harbormaster completed remote builds in B1543: Diff 23604.Jul 5 2014, 10:49 PM
joshuaspence added a parent revision: D9840: Remove all edge events.Jul 6 2014, 10:20 PM
joshuaspence requested changes to this revision.Jul 7 2014, 12:00 AM
joshuaspence edited edge metadata.
joshuaspence added inline comments.
src/applications/transactions/storage/PhabricatorApplicationTransaction.php
599

As mentioned in D9837, I'm not convinced that there is a need to pass this in explicitly.

724

As above.

This revision now requires changes to proceed.Jul 7 2014, 12:00 AM
joshuaspence accepted this revision.Jul 7 2014, 4:23 AM
joshuaspence edited edge metadata.
This revision is now accepted and ready to land.Jul 7 2014, 4:23 AM
epriestley updated this revision to Diff 23698.Jul 10 2014, 12:26 PM
epriestley edited edge metadata.
  • getByConst() -> getByConstant()
Harbormaster completed remote builds in B1605: Diff 23698.Jul 10 2014, 12:28 PM
epriestley closed this revision.Jul 17 2014, 10:41 PM
epriestley updated this revision to Diff 23929.

Closed by commit rPb32313cc85d8 (authored by @epriestley).

Revision Contents
Changeset List

PathSize
src/
applications/
transactions/
storage/
63 lines

Diff 23604

View Options

src/applications/transactions/storage/PhabricatorApplicationTransaction.php

Loading...
Phabricator · Built by Phacility