Skip to content

Bump meilisearch-go#1519

Closed
karjo24 wants to merge 2 commits into
devfrom
enh/bump-meili-search
Closed

Bump meilisearch-go#1519
karjo24 wants to merge 2 commits into
devfrom
enh/bump-meili-search

Conversation

@karjo24
Copy link
Copy Markdown
Contributor

@karjo24 karjo24 commented Mar 14, 2025

Motivation and Context

This PR is in preparation for adding the new custom lecture titles (#1510) to Meilisearch. To be able to get a single list for lectures both based on lecturer-given and user-defined titles, we need the federated search feature of Meilisearch, which the current version of meilisearch-go doesn't support.

Steps for Testing

Prerequisites:

  • 1 running instance of Meilisearch, e.g. using
docker run -it --rm \
    -p 7700:7700 \
    -e MEILI_MASTER_KEY='MASTER_KEY'\
    -v $(pwd)/meili_data:/meili_data \
    getmeili/meilisearch:v1.13.0 \
    meilisearch --env="development"

Without any guarantees, that this doesn't break anything: Note that if you had Meilisearch on a lower version running on your device before, this may trigger an incompatible indices error. To avoid having to migrate the databases, just delete the working directory of meilisearch (here meili_data), start Meilisearch again and then simply run the Cron job Export to Meili from TUMLive's admin->maintenance panel.

Steps for Testing:
Make sure that the search function works just as before. This PR doesn't change our search requests, only updates the code to work with the newer version.

@karjo24 karjo24 self-assigned this Mar 14, 2025
@karjo24 karjo24 added dependencies Pull requests that update a dependency file go Pull requests that update Go code labels Mar 16, 2025
@karjo24
Copy link
Copy Markdown
Contributor Author

karjo24 commented Mar 18, 2025

Closed as subset of #1462.

@karjo24 karjo24 closed this Mar 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file go Pull requests that update Go code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant