Differential D20852

Support more than 9 portals
ClosedPublic
Actions

Authored by artms on Oct 2 2019, 7:31 PM.

Tags

None

Referenced Files

	F15548197: D20852.diff
	Sat, Apr 26, 10:25 PM

	F15509128: D20852.id49711.diff
	Wed, Apr 16, 9:49 AM

	F15505331: D20852.id.diff
	Tue, Apr 15, 2:19 AM

	F15496188: D20852.id49709.diff
	Sun, Apr 13, 6:16 AM

	F15492963: D20852.id49711.diff
	Sat, Apr 12, 8:10 PM

	F15487345: D20852.id49711.diff
	Thu, Apr 10, 1:19 PM

	F15464670: D20852.id.diff
	Wed, Apr 2, 12:09 PM

	F15463368: D20852.diff
	Tue, Apr 1, 10:50 PM

View All Files

Subscribers

Korvin

Details

Reviewers

epriestley
Pawka

Group Reviewers

Blessed Reviewers

Commits

rP960c447aab7f: Support more than 9 portals

Summary

If portal is created with id > 9 - then those portals are not reachable and always return 404.

Test Plan

Create at least 10 portals, 10th and rest of them will no longer return 404.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

artms created this revision.Oct 2 2019, 7:31 PM

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptOct 2 2019, 7:31 PM

Harbormaster completed remote builds in B23537: Diff 49709.Oct 2 2019, 7:32 PM

artms requested review of this revision.Oct 2 2019, 7:32 PM

Herald added a subscriber: Korvin. · View Herald TranscriptOct 2 2019, 7:32 PM

Thanks!

This revision is now accepted and ready to land.Oct 2 2019, 7:33 PM

Closed by commit rP960c447aab7f: Support more than 9 portals (authored by artms). · Explain WhyOct 2 2019, 7:36 PM

This revision was automatically updated to reflect the committed changes.

artms added a commit: rP960c447aab7f: Support more than 9 portals.

Revision Contents
Changeset List

Path

Size

src/

applications/

dashboard/

application/

PhabricatorDashboardApplication.php

2 lines

Diff 49711

View Options

src/applications/dashboard/application/PhabricatorDashboardApplication.php

Support more than 9 portalsClosedPublicActions