Page MenuHomePhabricator
Differential D17090

Add Ngram support to Dashboards / Panels
ClosedPublic
Actions

Authored by chad on Dec 16 2016, 6:39 PM.
Tags
None
Referenced Files
F15430122: D17090.id41105.diff
Mon, Mar 24, 5:46 AM
F15427107: D17090.id.diff
Sun, Mar 23, 12:53 PM
F15331829: D17090.id41105.diff
Fri, Mar 7, 2:10 PM
F15331663: D17090.diff
Fri, Mar 7, 12:30 PM
F15302686: D17090.id41107.diff
Wed, Mar 5, 7:54 PM
Unknown Object (File)
Feb 26 2025, 5:23 PM
Unknown Object (File)
Feb 18 2025, 12:15 PM
Unknown Object (File)
Feb 9 2025, 3:02 AM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Commits
rPc6bdd2c56bab: Add Ngram support to Dashboards / Panels
Summary

Build ngram indexs, adds search by name capability.

Test Plan

Search for a dashboard by partial name, search for a panel by partial name.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad created this revision.Dec 16 2016, 6:39 PM
Herald added a subscriber: Korvin. · View Herald TranscriptDec 16 2016, 6:39 PM
epriestley accepted this revision.Dec 16 2016, 8:06 PM
epriestley edited edge metadata.
epriestley added inline comments.
src/applications/dashboard/query/PhabricatorDashboardSearchEngine.php
24

"badges" should be "dashboards".

This revision is now accepted and ready to land.Dec 16 2016, 8:06 PM
chad added inline comments.Dec 16 2016, 8:06 PM
src/applications/dashboard/query/PhabricatorDashboardPanelSearchEngine.php
54

this one's ok though? amirite?

chad added a comment.Dec 16 2016, 8:07 PM

daftpunk

Closed by commit rPc6bdd2c56bab: Add Ngram support to Dashboards / Panels (authored by chad, committed by chad). · Explain WhyDec 16 2016, 8:09 PM
This revision was automatically updated to reflect the committed changes.
epriestley added inline comments.Dec 16 2016, 8:09 PM
src/applications/dashboard/query/PhabricatorDashboardPanelSearchEngine.php
54

ACTUALLY IT LOOKS LIKE IT IS INDENTED TOO MUCH

frowncat

epriestley added a comment.Dec 16 2016, 8:13 PM

I am also shocked that you apparently encountered 0 fatals/exceptions/bugs while creating and landing this diff instead of the expected 50+.

Revision Contents
Changeset List

Diff 41107

View Options

resources/sql/autopatches/20161216.dashboard.ngram.01.sql

Loading...
View Options

resources/sql/autopatches/20161216.dashboard.ngram.02.php

Loading...
View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/dashboard/editor/PhabricatorDashboardPanelEditEngine.php

Loading...
View Options

src/applications/dashboard/query/PhabricatorDashboardPanelQuery.php

Loading...
View Options

src/applications/dashboard/query/PhabricatorDashboardPanelSearchEngine.php

Loading...
View Options

src/applications/dashboard/query/PhabricatorDashboardQuery.php

Loading...
View Options

src/applications/dashboard/query/PhabricatorDashboardSearchEngine.php

Loading...
View Options

src/applications/dashboard/storage/PhabricatorDashboard.php

Loading...
View Options

src/applications/dashboard/storage/PhabricatorDashboardNgrams.php

Loading...
View Options

src/applications/dashboard/storage/PhabricatorDashboardPanel.php

Loading...
View Options

src/applications/dashboard/storage/PhabricatorDashboardPanelNgrams.php

Loading...
Phabricator · Built by Phacility