Page MenuHomePhabricator
Differential D16394

Make "Core Applications" more reasonable
ClosedPublic
Actions

Authored by chad on Aug 12 2016, 3:06 AM.
Tags
None
Referenced Files
F15533559: D16394.id39426.diff
Wed, Apr 23, 8:26 PM
F15422599: D16394.id39427.diff
Mar 22 2025, 9:43 AM
F15422050: D16394.diff
Mar 22 2025, 5:01 AM
F15421554: D16394.id39426.diff
Mar 22 2025, 12:46 AM
F15367484: D16394.diff
Mar 11 2025, 6:53 PM
F15330468: D16394.id39427.diff
Mar 7 2025, 7:42 AM
Unknown Object (File)
Feb 25 2025, 9:51 PM
Unknown Object (File)
Feb 18 2025, 11:35 AM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T11132: New Phabricator NUX
Commits
rPf3d8e3832c96: (stable) Promote 2016 Week 33
rPde09334a364f: (stable) Promote 2016 Week 33
rPfabb716fbcd9: Merge branch 'master' of ssh://secure.phabricator.com/diffusion/P
rP65e964fca14d: Make "Core Applications" more reasonable
Summary

Ref T11132, cleaning up what "Core Applications" means.

Test Plan

Visit /applications/, see less poseurs.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad updated this revision to Diff 39426.Aug 12 2016, 3:06 AM
chad retitled this revision from to Make "Core Applications" more reasonable.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
chad added a task: T11132: New Phabricator NUX.
Herald added a subscriber: Korvin. · View Herald TranscriptAug 12 2016, 3:06 AM
chad added a comment.Aug 12 2016, 4:11 AM

I'd like to see core apps as the default left sidebar on home. When admin comes in and removes a core app, it's then removed from default home. I think this is simple to understand.

epriestley accepted this revision.Aug 12 2016, 1:58 PM
epriestley edited edge metadata.

(Uninstalling an application should already remove it.)

This revision is now accepted and ready to land.Aug 12 2016, 1:58 PM
Closed by commit rP65e964fca14d: Make "Core Applications" more reasonable (authored by chad, committed by chad). · Explain WhyAug 12 2016, 2:57 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
calendar/
application/
4 lines
feed/
application/
4 lines
packages/
application/
4 lines
people/
application/
4 lines
phurl/
application/
4 lines
ponder/
application/
4 lines
releeph/
application/
4 lines

Diff 39427

View Options

src/applications/calendar/application/PhabricatorCalendarApplication.php

Loading...
View Options

src/applications/feed/application/PhabricatorFeedApplication.php

Loading...
View Options

src/applications/packages/application/PhabricatorPackagesApplication.php

Loading...
View Options

src/applications/people/application/PhabricatorPeopleApplication.php

Loading...
View Options

src/applications/phurl/application/PhabricatorPhurlApplication.php

Loading...
View Options

src/applications/ponder/application/PhabricatorPonderApplication.php

Loading...
View Options

src/applications/releeph/application/PhabricatorReleephApplication.php

Loading...
Phabricator · Built by Phacility