ClosedPublic
Actions

Authored by chad on Feb 23 2015, 8:43 PM.

Tags

None

Referenced Files

	F15480421: D11868.id28584.diff
	Tue, Apr 8, 2:01 PM

	F15458984: D11868.id28584.diff
	Mon, Mar 31, 8:55 AM

	F15421731: D11868.id.diff
	Mar 22 2025, 1:55 AM

	F15418385: D11868.diff
	Mar 20 2025, 10:59 PM

	F15413081: D11868.id28584.diff
	Mar 19 2025, 4:59 PM

	Unknown Object (File)
	Mar 3 2025, 7:12 AM

	Unknown Object (File)
	Feb 10 2025, 1:00 AM

	Unknown Object (File)
	Feb 8 2025, 2:42 AM

View All Files

Subscribers

epriestley

Korvin

Details

Reviewers

epriestley
btrahan

Commits

Restricted Diffusion Commit
rP2518a5eaff2b: Fix exception on /project/

Summary

I missed testing this case when re-doing the navigation.

Test Plan

Test /project/, don't see fatal. Test mobile menu, works as expected.

Diff Detail

Repository

rP Phabricator

Branch

project-oops

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 4622
Build 4636: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

chad updated this revision to Diff 28584.Feb 23 2015, 8:43 PM

chad retitled this revision from to Fix exception on /project/.

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 23 2015, 8:43 PM

epriestley accepted this revision.Feb 23 2015, 8:45 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Feb 23 2015, 8:45 PM

Closed by commit rP2518a5eaff2b: Fix exception on /project/ (authored by chad, committed by chad). · Explain WhyFeb 23 2015, 8:46 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

project/

controller/

PhabricatorProjectController.php

1 line

Diff 28584

View Options

src/applications/project/controller/PhabricatorProjectController.php

Fix exception on /project/ClosedPublicActions