Page MenuHomePhabricator

Allow Home and Dashboards to be uninstalled
ClosedPublic

Authored by epriestley on Feb 11 2015, 10:57 PM.
Tags
None
Referenced Files
F14038181: D11753.id28333.diff
Sun, Nov 10, 10:02 PM
F14038158: D11753.diff
Sun, Nov 10, 9:51 PM
F13990940: D11753.diff
Tue, Oct 22, 7:29 AM
F13990930: D11753.id.diff
Tue, Oct 22, 7:24 AM
F13990927: D11753.id28333.diff
Tue, Oct 22, 7:23 AM
F13990922: D11753.id28336.diff
Tue, Oct 22, 7:21 AM
F13976299: D11753.diff
Oct 18 2024, 1:39 PM
F13971806: D11753.id28333.diff
Oct 17 2024, 3:25 PM
Subscribers

Details

Summary

Ref T7143. This is the simplest fix for adding a new route for Home, at the cost of possibly letting users break instances. However:

  • It's kind of hard to get to the option to uninstall Home anyway.
  • It's hard to imagine anyone will really uninstall Home by accident, right? Right?
  • Put a really scary warning on the action just in case.

Dashboards was only required because Home was required, I think, so just drop that too.

Test Plan

Uninstalled home.

Diff Detail

Repository
rP Phabricator
Branch
rmhome
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 4473
Build 4487: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley retitled this revision from to Allow Home and Dashboards to be uninstalled.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.

Even if you do this, you can still reinstall the application, you just need to get back to Applications somehow (e.g., back button or search or typing in the URL).

If users do manage to do this by accident, we could special case the "/" route if no controller matches it and print out some kind of "go reinstall Home" thing, but that was a bit of a mess and didn't feel very general so I figured we could wait until someone manages to do this.

btrahan edited edge metadata.
This revision is now accepted and ready to land.Feb 11 2015, 11:20 PM

It's hard to imagine anyone will really uninstall Home by accident, right? Right?

#smh

This revision was automatically updated to reflect the committed changes.