Page MenuHomePhabricator
Create Task
Maniphest T5193

Unable to install Releeph application
Closed, ResolvedPublic
Actions

Description

I am unable to install the "Releeph" application on our Phabricator install. If I go to the /applications/view/PhabricatorApplicationReleeph/ endpoint and click on the "Install" button then I am presented with a confirmation modal. After clicking "Install" in the modal, I am taken back to the https://phabricator.freelancer.com/applications/view/PhabricatorApplicationReleeph/ URL, but Releeph is still uninstalled.

There are no errors in the PHP error logs.

Related Objects

Event Timeline

joshuaspence created this task.May 27 2014, 11:13 AM
joshuaspence assigned this task to epriestley.
joshuaspence raised the priority of this task from to Needs Triage.
joshuaspence updated the task description. (Show Details)
joshuaspence added a project: Releeph.
joshuaspence added a subscriber: joshuaspence.
epriestley added a comment.May 27 2014, 12:43 PM

This should work in a more clear way, but the root cause is likely that releeph.installed is off.

joshuaspence added a comment.May 27 2014, 12:50 PM

That seems logical.

epriestley added a comment.May 27 2014, 1:22 PM

◀ Merged tasks: T5191.

chad triaged this task as Normal priority.May 30 2014, 2:34 AM
joshuaspence added a comment.Jun 3 2014, 7:51 PM

I think that the correct fix here is to remove the releeph.installed configuration option and to, instead, allow installation of Releeph from /applications/view/PhabricatorApplicationReleeph/ as with all other applications.

Thoughts?

epriestley added a comment.Jun 3 2014, 7:59 PM

There's no way to make an application default to "uninstalled" right now. We could build such a mechanism, but I'm not sure it would be very useful in general. Releeph is unusual in that it's production-ready for Facebook only, so the reasonable tricks (like delisting pre-beta applications) we use for other applications (like Dashboards) aren't great fits.

I think a better solution is probably:

  • Do nothing for now;
  • move Releeph to beta eventually (T2714) and remove releeph.installed;
  • not do this again in the future; and
  • if more users run into confusion before T2714 lands, maybe add a big IMPORTANT: to the application page? We have a mechanism for this now.
joshuaspence added a comment.Jun 4 2014, 7:06 PM

Yeah sure, fair call.

epriestley closed this task as Resolved.Oct 3 2014, 4:10 AM

We renamed these to "Prototype" applications with stricter warnings about quality and support, and made Releeph a normal prototype.

Phabricator · Built by Phacility