Skip to content

2361 improve search#12296

Closed
PothieuG wants to merge 2 commits intomainfrom
2361-Improve_Search
Closed

2361 improve search#12296
PothieuG wants to merge 2 commits intomainfrom
2361-Improve_Search

Conversation

@PothieuG
Copy link
Copy Markdown
Member

@PothieuG PothieuG commented Mar 24, 2026

Description

PBI - SSWConsulting/SSW.Rules#2361
Related PR - SSWConsulting/SSW.Rules#2544

Note:

  • I had to update Algolia index shape (with content for instance)
  • I had to limit the content size so it would fit within the indexing JSON file's size limit due to limitation of our Algolia free plan.
    Most of the content for each file are in it though, but for very large rule, we could have a limitation

Screenshot (optional)

Image

Figure: now the search is looking for content and also put some highlights

@github-actions github-actions Bot added Age: 🥚 - New About 2 hours old Age: 🐣 - Young About 4 hours old Age: 🐥 - Adolescent About 8 hours old and removed Age: 🥚 - New About 2 hours old Age: 🐣 - Young About 4 hours old labels Mar 24, 2026
@tiagov8
Copy link
Copy Markdown
Member

tiagov8 commented Mar 24, 2026

Hi @PothieuG

  1. Please update the description
  2. Fix the conflicts
  3. Get a dev to review

@github-actions github-actions Bot added Age: 🐤 - Mature About 16 hours old and removed Age: 🐥 - Adolescent About 8 hours old labels Mar 25, 2026
@PothieuG PothieuG closed this Mar 25, 2026
@PothieuG PothieuG reopened this Mar 25, 2026
@github-actions github-actions Bot added Age: 🐓 - Old About 32 hours old and removed Age: 🐤 - Mature About 16 hours old labels Mar 25, 2026
@tiagov8 tiagov8 requested a review from Aibono1225 March 25, 2026 20:51
@github-actions github-actions Bot added Age: 🍗 - Ancient About 64 hours old 🔥 Merge Debt This PR contains merge debt, see https://www.ssw.com.au/rules/merge-debt/ and removed Age: 🐓 - Old About 32 hours old labels Mar 26, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Howzit @PothieuG,

This PR has been here a while.

Did you know you should avoid merge debt?

  1. Please action (e.g. get a review) and merge or close

Thanks!

@Aibono1225 Aibono1225 closed this Mar 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Age: 🍗 - Ancient About 64 hours old 🔥 Merge Debt This PR contains merge debt, see https://www.ssw.com.au/rules/merge-debt/

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants