Page MenuHomePhabricator
Differential D11223

T6597, Uninstalled applications are still linked on Application launcher
ClosedPublic
Actions

Authored by lpriestley on Jan 5 2015, 8:07 PM.
Tags
None
Referenced Files
F15440344: D11223.id.diff
Wed, Mar 26, 12:20 PM
F15439717: D11223.id26949.diff
Wed, Mar 26, 9:03 AM
F15439288: D11223.id26968.diff
Wed, Mar 26, 6:47 AM
F15439280: D11223.id26964.diff
Wed, Mar 26, 6:44 AM
F15438423: D11223.diff
Wed, Mar 26, 12:35 AM
F15413803: D11223.id26968.diff
Wed, Mar 19, 9:27 PM
F15398382: D11223.id26964.diff
Mon, Mar 17, 12:03 AM
F15390426: D11223.id.diff
Sat, Mar 15, 6:12 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T6597: Uninstalled applications are still linked on Application launcher
Commits
Restricted Diffusion Commit
rP87c1d004be02: T6597, Uninstalled applications are still linked on Application launcher
Summary

Fixes T6597, Uninstalled applications should not be clickable when searching "All Applications" in the Applications launcher

Test Plan

Navigate too /applications/query/all, uninstall an application, navigate back to all applications. Uninstalled application title should not be clickable.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

lpriestley updated this revision to Diff 26949.Jan 5 2015, 8:07 PM
lpriestley retitled this revision from to T6597, Uninstalled applications are still linked on Application launcher.
lpriestley updated this object.
lpriestley edited the test plan for this revision. (Show Details)
lpriestley added a reviewer: epriestley.
lpriestley added a task: T6597: Uninstalled applications are still linked on Application launcher.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJan 5 2015, 8:07 PM
Herald added subscribers: epriestley, Korvin. · View Herald Transcript
lpriestley updated this revision to Diff 26964.Jan 5 2015, 11:41 PM
lpriestley edited edge metadata.

cleaning up

epriestley accepted this revision.Jan 6 2015, 12:25 AM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jan 6 2015, 12:25 AM
Closed by commit rP87c1d004be02: T6597, Uninstalled applications are still linked on Application launcher (authored by lpriestley, committed by epriestley). · Explain WhyJan 6 2015, 12:25 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
meta/
query/
2 lines

Diff 26968

View Options

src/applications/meta/query/PhabricatorAppSearchEngine.php

Loading...
Phabricator · Built by Phacility