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
F14060283: D11847.diff
Mon, Nov 18, 12:18 AM
F13968910: D11847.id28574.diff
Oct 16 2024, 11:25 PM
Unknown Object (File)
Oct 12 2024, 8:40 PM
Unknown Object (File)
Oct 1 2024, 5:07 AM
Unknown Object (File)
Sep 25 2024, 12:52 PM
Unknown Object (File)
Sep 17 2024, 10:57 PM
Unknown Object (File)
Sep 2 2024, 8:10 AM
Unknown Object (File)
Aug 28 2024, 9:48 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