Use new menu constants in home menu item
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F18082684: D17247.id41483.diff
	Tue, Aug 5, 7:04 AM

	F17952180: D17247.id.diff
	Fri, Aug 1, 1:59 AM

	F17942359: D17247.id41482.diff
	Thu, Jul 31, 6:41 AM

	F17941513: D17247.diff
	Thu, Jul 31, 5:32 AM

	F17928334: D17247.diff
	Wed, Jul 30, 4:03 PM

	Unknown Object (File)
	Jun 19 2025, 12:35 PM

	Unknown Object (File)
	Jun 18 2025, 12:41 AM

	Unknown Object (File)
	Jun 17 2025, 5:55 PM

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