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
F15411720: D13850.diff
Wed, Mar 19, 9:37 AM
F15407803: D13850.id33439.diff
Tue, Mar 18, 7:41 PM
F15385907: D13850.diff
Fri, Mar 14, 11:38 PM
F15333571: D13850.id.diff
Sat, Mar 8, 3:09 AM
F15332539: D13850.id33439.diff
Fri, Mar 7, 5:47 PM
F15331419: D13850.id33455.diff
Fri, Mar 7, 10:15 AM
Unknown Object (File)
Sat, Feb 22, 4:27 PM
Unknown Object (File)
Feb 8 2025, 9:54 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