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
F13245042: D9841.diff
Thu, May 23, 5:35 AM
F13236899: D9841.id23698.diff
Tue, May 21, 11:25 AM
F13216617: D9841.id23604.diff
Sat, May 18, 1:35 AM
F13208151: D9841.id23698.diff
Thu, May 16, 10:07 AM
F13195776: D9841.id.diff
Sun, May 12, 10:37 PM
F13194527: D9841.diff
Sun, May 12, 9:38 PM
F13177951: D9841.diff
Wed, May 8, 8:04 PM
Unknown Object (File)
Sun, May 5, 1:27 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
Lint
Lint Skipped
Unit
Tests Skipped

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–600

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 23929

View Options

src/applications/transactions/storage/PhabricatorApplicationTransaction.php

Loading...
Phabricator · Built by Phacility