8000 feat: sqlite fts based search by 18alantom · Pull Request #398 · frappe/wiki · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat: sqlite fts based search #398

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
May 21, 2025
Merged

Conversation

18alantom
Copy link
Member
@18alantom 18alantom commented May 21, 2025

i needed a playground (in prod) for some search ideas i need for another project i'm working on. search is based on SQLite FTS5 cause it serves as a decent stepping board. added some minor reranking over returned results.

baseline has better (anecdotal) results compared to existing (redis based) search. added minor ux improvements (reduced debounce).

Copy link
cypress bot commented May 21, 2025

Wiki    Run #510

Run Properties:  status check failed Failed #510  •  git commit 28c0976d75 ℹ️: Merge 83da2d9a61a14423c4329854b13d123734f79180 into e5895af94e14ac83d355ea90d8fd...
Project Wiki
Branch Review sqlite-fts
Run status status check failed Failed #510
Run duration 01m 21s
Commit git commit 28c0976d75 ℹ️: Merge 83da2d9a61a14423c4329854b13d123734f79180 into e5895af94e14ac83d355ea90d8fd...
Committer Alan
View all properties for this run ↗︎

Test results
Tests that failed  Failures 5
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 0
View all changes introduced in this branch ↗︎

Tests for review

Failed  wiki.cy.js • 3 failed tests

View Output

Test Artifacts
Wiki > creates a new wiki page Test Replay Screenshots
Wiki > edits a wiki page Test Replay Screenshots
Wiki > deletes a wiki page Test Replay Screenshots
Failed  wiki_sidebar.cy.js • 2 failed tests

View Output

Test Artifacts
Wiki Sidebar > creates a new wiki sidebar group Screenshots
Wiki Sidebar > deletes a wiki sidebar group when the group is empty Screenshots

@18alantom
Copy link
Member Author
18alantom commented May 21, 2025

cypress tests have failed cause of timeouts. linter appears to have failed cause of files untouched in this PR.

@18alantom 18alantom merged commit fdc54b1 into frappe:master May 21, 2025
4 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant
0