Page MenuHomePhabricator
Differential D15457

Move DifferentialRevisionViewController to newPage()
ClosedPublic
Actions

Authored by epriestley on Mar 10 2016, 9:20 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Sep 9, 4:12 PM
Unknown Object (File)
Tue, Sep 3, 4:17 PM
Unknown Object (File)
Mon, Sep 2, 10:24 PM
Unknown Object (File)
Wed, Aug 28, 6:29 PM
Unknown Object (File)
Aug 8 2024, 3:57 AM
Unknown Object (File)
Aug 7 2024, 3:57 AM
Unknown Object (File)
Aug 3 2024, 3:35 AM
Unknown Object (File)
Jul 30 2024, 3:45 AM
View All Files
Subscribers
None

Details

Reviewers
chad
Commits
Restricted Diffusion Commit
rP7b8da9991429: Move DifferentialRevisionViewController to newPage()
Summary

I think this works?

Test Plan

i am wizard

Screen Shot 2016-03-10 at 1.19.22 PM.png (1×3 px, 389 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 37251.Mar 10 2016, 9:20 PM
epriestley retitled this revision from to Move DifferentialRevisionViewController to newPage().
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
Herald edited edge metadata. · View Herald TranscriptMar 10 2016, 9:20 PM
chad added a comment.Mar 10 2016, 9:21 PM

haha, there already was a setNavigation lulz.

epriestley added a comment.Mar 10 2016, 9:21 PM

i am time wizard

chad accepted this revision.Mar 10 2016, 9:22 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Mar 10 2016, 9:22 PM
Closed by commit rP7b8da9991429: Move DifferentialRevisionViewController to newPage() (authored by epriestley, committed by epriestley). · Explain WhyMar 10 2016, 9:22 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
differential/
controller/
22 lines

Diff 37252

View Options

src/applications/differential/controller/DifferentialRevisionViewController.php

Loading...
Phabricator · Built by Phacility