Page MenuHomePhabricator
Differential D13850

Read Differential changeset coverage information from Harbormaster
ClosedPublic
Actions

Authored by epriestley on Aug 10 2015, 8:36 PM.
Tags
None
Referenced Files
F18103295: D13850.id33439.diff
Sun, Aug 10, 3:51 AM
F18085046: D13850.diff
Tue, Aug 5, 6:58 PM
F18047613: D13850.id33455.diff
Sun, Aug 3, 12:39 PM
F17843461: D13850.id.diff
Sun, Jul 27, 12:55 AM
F17805321: D13850.diff
Fri, Jul 25, 12:27 PM
F17707462: D13850.diff
Wed, Jul 16, 3:04 PM
Unknown Object (File)
Jun 17 2025, 11:35 PM
Unknown Object (File)
Jun 16 2025, 12:35 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
rPefa8855e0308: Read Differential changeset coverage information from Harbormaster
Summary

Ref T8096. This information has moved into Harbormaster.

Test Plan

Pushed some test results in; see right margin:

Screen Shot 2015-08-10 at 1.35.39 PM.png (416×869 px, 57 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 33439.Aug 10 2015, 8:36 PM
epriestley retitled this revision from to Read Differential changeset coverage information 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).
joshuaspence awarded a token.Aug 10 2015, 8:36 PM
epriestley added a comment.Aug 10 2015, 8:37 PM

Oops, this screenshot actually shows the feature working:

Screen Shot 2015-08-10 at 1.35.39 PM.png (416×869 px, 57 KB)

epriestley edited the test plan for this revision. (Show Details)Aug 10 2015, 8:37 PM
chad accepted this revision.Aug 10 2015, 8:58 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Aug 10 2015, 8:58 PM
Closed by commit rPefa8855e0308: Read Differential changeset coverage information 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/
104 lines
customfield/
19 lines
storage/
16 lines

Diff 33455

View Options

src/applications/differential/controller/DifferentialChangesetViewController.php

Loading...
View Options

src/applications/differential/customfield/DifferentialHarbormasterField.php

Loading...
View Options

src/applications/differential/storage/DifferentialDiff.php

Loading...
Phabricator · Built by Phacility