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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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

Path

Size

src/

applications/

people/

controller/

PhabricatorPeopleController.php

4 lines

Diff 28574

View Options

src/applications/people/controller/PhabricatorPeopleController.php

Fix People mobile menu URLsClosedPublicActions