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.

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

Migrate to preCICE Algolia account, update index in a workflow #408

Merged
merged 3 commits into from
May 22, 2024

Conversation

MakisH
Copy link
Member

@MakisH MakisH commented May 13, 2024

@chlorenz I updated the _config.yml, is anything else needed to use the new index?

@fsimonis I also added a workflow to update the search index every Sunday at 16:00 (or manually), due to which, this solves #237.

I have not tested the Action.

@MakisH MakisH requested a review from chlorenz May 13, 2024 14:26
@MakisH MakisH self-assigned this May 13, 2024
@MakisH MakisH changed the title Migrate to preCICE Algolia account Migrate to preCICE Algolia account, update index in a workflow May 13, 2024
@MakisH
Copy link
Member Author

MakisH commented May 22, 2024

@chlorenz can I merge this already, or did you want to have a look first on whether I forgot anything?

@chlorenz
Copy link
Collaborator

Seems all good. If I remember correctly when you run the algolia gem the first time it should build the index from scratch and everything should be as before the migration. Only downside I can see is that we lose the continuity in analytics, but that's definitely worth it.

@MakisH MakisH merged commit 07a6403 into precice:master May 22, 2024
2 checks passed
@MakisH MakisH deleted the update-algolia branch May 22, 2024 17:15
carme-hp pushed a commit that referenced this pull request May 23, 2024
* Migrate to preCICE Algolia account
* Add manual workflow to update the Algolia index
* Add cron schedule
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.

2 participants