Page MenuHomePhabricator
Differential D11847

Fix People mobile menu URLs
ClosedPublic
Actions

Authored by chad on Feb 21 2015, 2:46 PM.
Tags
None
Referenced Files
F13153266: D11847.diff
Sun, May 5, 1:53 AM
Unknown Object (File)
Fri, May 3, 3:55 AM
Unknown Object (File)
Mon, Apr 29, 4:04 PM
Unknown Object (File)
Thu, Apr 25, 8:48 PM
Unknown Object (File)
Wed, Apr 24, 11:52 PM
Unknown Object (File)
Tue, Apr 16, 2:48 PM
Unknown Object (File)
Thu, Apr 11, 8:09 AM
Unknown Object (File)
Mar 31 2024, 5:53 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
btrahan
Commits
Restricted Diffusion Commit
rP15824bd51676: Fix People mobile menu URLs
Summary

The mobile menu on people profiles has the incorrect order in the URLs and thus, 404s.

Test Plan

Went to a profile on a mobile display, click on feed and calendar links, got to correct place.

Diff Detail

Repository
rP Phabricator
Branch
people-mobile-menu
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 4597
Build 4611: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

chad updated this revision to Diff 28551.Feb 21 2015, 2:46 PM
chad retitled this revision from to Fix People mobile menu URLs.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added reviewers: epriestley, btrahan.
Herald added subscribers: epriestley, Korvin. · View Herald TranscriptFeb 21 2015, 2:46 PM
btrahan accepted this revision.Feb 23 2015, 5:16 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Feb 23 2015, 5:16 PM
Closed by commit rP15824bd51676: Fix People mobile menu URLs (authored by chad, committed by chad). · Explain WhyFeb 23 2015, 5:23 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
people/
controller/
4 lines

Diff 28551

View Options

src/applications/people/controller/PhabricatorPeopleController.php

Loading...
Phabricator · Built by Phacility