Page MenuHomePhabricator
Differential D13847

Read synthetic inline comments from Harbormaster
ClosedPublic
Actions

Authored by epriestley on Aug 10 2015, 7:26 PM.
Tags
None
Referenced Files
F18771559: D13847.id33435.diff
Wed, Oct 8, 8:20 PM
F18751019: D13847.id.diff
Sat, Oct 4, 9:32 AM
F18745261: D13847.diff
Fri, Oct 3, 5:56 AM
F18731341: D13847.id33453.diff
Tue, Sep 30, 2:33 PM
F18712672: D13847.id.diff
Mon, Sep 29, 5:30 AM
F18621383: D13847.diff
Sep 15 2025, 8:03 AM
F18595488: D13847.diff
Sep 12 2025, 9:56 PM
F18594265: D13847.id33453.diff
Sep 12 2025, 5:36 PM
View All Files
Subscribers
None
Tokens
"Doubloon" token, awarded by joshuaspence.

Details

Reviewers
chad
Maniphest Tasks
T8096: Improve Harbormaster UI/UX (v1)
Commits
Restricted Diffusion Commit
rP2b13da88ceb9: Read synthetic inline comments from Harbormaster
Summary

Ref T8096. These are still reading out of the old diff property, but should read from Harbormaster instead.

Test Plan

Screen Shot 2015-08-10 at 12.24.45 PM.png (179×879 px, 23 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 33435.Aug 10 2015, 7:26 PM
epriestley retitled this revision from to Read synthetic inline comments from Harbormaster.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T8096: Improve Harbormaster UI/UX (v1).
chad accepted this revision.Aug 10 2015, 7:42 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Aug 10 2015, 7:42 PM
joshuaspence awarded a token.Aug 10 2015, 8:35 PM
Closed by commit rP2b13da88ceb9: Read synthetic inline comments from Harbormaster (authored by epriestley, committed by epriestley). · Explain WhyAug 10 2015, 9:16 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
differential/
controller/
76 lines

Diff 33453

View Options

src/applications/differential/controller/DifferentialChangesetViewController.php

Loading...
Phabricator · Built by Phacility