Use new menu constants in home menu item
ClosedPublic
Actions

Authored by chad on Jan 25 2017, 5:07 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Dec 21, 9:46 AM

	Unknown Object (File)
	Tue, Dec 17, 12:57 PM

	Unknown Object (File)
	Mon, Dec 9, 7:03 AM

	Unknown Object (File)
	Mon, Dec 9, 6:59 AM

	Unknown Object (File)
	Mon, Dec 9, 3:46 AM

	Unknown Object (File)
	Mon, Dec 9, 3:33 AM

	Unknown Object (File)
	Sun, Dec 8, 11:56 PM

	Unknown Object (File)
	Fri, Dec 6, 9:02 AM

View All Files

Subscribers

Korvin

Details

Reviewers

epriestley

Maniphest Tasks

T11957: Add MenuEngine to Home

Commits

rPce09ab9b0e1a: Use new menu contsants in home menu item

Summary

Ref T11957, just lays in some minor bug fixes. Sets correct menu, removes sidebar on edit.

Test Plan

Test /menu/ on home with Admin and Normal accounts.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

chad created this revision.Jan 25 2017, 5:07 PM

Herald added a subscriber: Korvin. · View Herald TranscriptJan 25 2017, 5:07 PM

Harbormaster completed remote builds in B15372: Diff 41482.Jan 25 2017, 5:08 PM

epriestley accepted this revision.Jan 25 2017, 5:10 PM

This revision is now accepted and ready to land.Jan 25 2017, 5:10 PM

chad retitled this revision from Use new menu contants in home menu item to Use new menu contsants in home menu item.Jan 25 2017, 5:21 PM

Closed by commit rPce09ab9b0e1a: Use new menu contsants in home menu item (authored by chad, committed by chad). · Explain WhyJan 25 2017, 5:22 PM

This revision was automatically updated to reflect the committed changes.

super important

Revision Contents
Changeset List

Path

Size

src/

applications/

home/

controller/

PhabricatorHomeMenuItemController.php

6 lines

Diff 41483

View Options

src/applications/home/controller/PhabricatorHomeMenuItemController.php

Use new menu constants in home menu itemClosedPublicActions