Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix(HelpdeskSearch): Improve search of knowledge base articles #1924

Merged
merged 13 commits into from
Aug 20, 2024
Merged

fix(HelpdeskSearch): Improve search of knowledge base articles #1924

merged 13 commits into from
Aug 20, 2024

Conversation

balamurali27
Copy link
Contributor

@balamurali27 balamurali27 commented Aug 14, 2024 •
edited by RitvikSardana
Loading

Before

image

After

image

To be done in UI

- [ ] Fix ticket id search Not doing this as it's too out of the way. Filter already exists for this anyway

  • Show headings instead of subject
  • Show description in search results. Render mark tags
  • Put # in url instead of %23
  • Replace the sql search in /my-tickets with HelpdeskSearch
  • Add permalinks to headings so urls will scroll to relevant section

balamurali27 and others added 10 commits August 14, 2024 12:44
@balamurali27
fix(HelpdeskSearch): Get better search results
6865c26 
- Search headings separately
- Reduce page length
- Change weights
- Summarize description in search results
@balamurali27
fix(HelpdeskSearch): Give permalinks to subsections
b8d9031 
- Change search algorithm to BM25. Seems to give better results
- Reduce fuzzyness as it seems to mess with accuracy for small words
  (ssh, site, row, etc)
@balamurali27
refactor(HelpdeskSearch): Remove unnecessary payload insertion
dfddd56
@balamurali27
chore(HelpdeskSearch): Fix commented out parts
97da939
@balamurali27 @github-actions
style(black): helpdesk/search.py
31ab113 
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@balamurali27
chore(HelpdeskSearch): Add back lost progressbar
82646f3
@balamurali27
fix(HelpdeskSearch): Don't do anything with payload
f7ae196
@balamurali27
fix(my-tickets): Show description list instead of just doc titles
323aa2e
@balamurali27
fix(my-tickets): Direct user to permalink instead of top of Article
8f8ccfe
@balamurali27
fix(HelpdeskSearch): Handle no payload returned in frontend
143ad49
@RitvikSardana RitvikSardana merged commit 8aa71cd into develop Aug 20, 2024
4 checks passed
@RitvikSardana RitvikSardana deleted the hd-search-fix branch December 10, 2024 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@RitvikSardana RitvikSardana RitvikSardana left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@balamurali27 @RitvikSardana