-
Notifications
You must be signed in to change notification settings - Fork 38
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
Feature: integrate meilisearch for searching #56
base: main
Are you sure you want to change the base?
Feature: integrate meilisearch for searching #56
Conversation
A new issue has been found! batching of data, as currently, it would attempt to dump the entire posts table to the index, however, the posts table has over 27K records sooooo. I'll need to batch the data, good thing is I already have code for it, just need to convert it over. |
…rSearch, and GetPostsForSearch
@previnder, the indexing logic along with everything else is finished. Now when pulling data to index it takes it in batches of 5000 records (I'll change it if you want it to be higher or lower, probably higher around 10,000 or 15,000). When pushing the data over to meilisearch we chunk it into batches of 50MiB (or less if we don't have that much data). By default, the searching ability is disabled and still makes use of the Google search to allow for the hoster to fully configure everything without having to worry about meilisearch till they are ready. |
…-meilisearch-for-search-endpoint
Adds support for the use of meilisearch for quick indexing and searching of discuit communities.
Required
If Possible
General
Need to do testing to ensure this will work correctly, from my own local testing everything works as expected. It also needs PR #49 merged so I can add docker support. And lastly after this is merged, someone who is better a UI stuff then me should probably make the searching actually be nice lol
Known Issues
Related Issues: