Skip to content

[v4] Update Algolia docsearch configuration #519

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
Merged
merged 2 commits into from
Nov 30, 2021
Merged

[v4] Update Algolia docsearch configuration #519

merged 2 commits into from
Nov 30, 2021

Conversation

Mcastres
Copy link
Contributor

What does it do?

Describe the technical changes you did.

Why is it needed?

Describe the issue you are solving.

Related issue(s)/PR(s)

Let us know if this is related to any issue/pull request

@Mcastres Mcastres added the flag: don't merge This PR should not be merged at the moment label Nov 30, 2021
@Mcastres Mcastres self-assigned this Nov 30, 2021
@vercel
Copy link

vercel bot commented Nov 30, 2021 •
edited
Loading

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployments, click below or on the icon next to each commit.

documentation – ./docs

🔍 Inspect: https://vercel.com/strapijs/documentation/Gws8Tusrgf7qFS35R49B8Nj5us7S
✅ Preview: https://documentation-git-algolia-update-config-strapijs.vercel.app

derrickmehaffy
derrickmehaffy previously approved these changes Nov 30, 2021
View reviewed changes
Copy link
Member

@derrickmehaffy derrickmehaffy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

pwizla
pwizla previously approved these changes Nov 30, 2021
View reviewed changes
Copy link
Collaborator

@pwizla pwizla left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks Maxime! 🥳

@pwizla pwizla changed the title Update config [v4] Update Algolia docsearch configuration Nov 30, 2021
@pwizla pwizla added source: repo PRs/issues not targeting a specific documentation but rather affecting the whole repo target: v4 Documentation PRs/issues targeting content from docs.strapi.io (main branch). pr: chore labels Nov 30, 2021
@Mcastres Mcastres dismissed stale reviews from pwizla and derrickmehaffy via e932d39 November 30, 2021 13:33
@derrickmehaffy derrickmehaffy marked this pull request as ready for review November 30, 2021 14:31
derrickmehaffy
derrickmehaffy approved these changes Nov 30, 2021
View reviewed changes
Copy link
Member

@derrickmehaffy derrickmehaffy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approved

@Mcastres Mcastres merged commit 9c041c3 into main Nov 30, 2021
@Mcastres Mcastres deleted the algolia/update-config branch November 30, 2021 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@derrickmehaffy derrickmehaffy derrickmehaffy approved these changes

@pwizla pwizla pwizla left review comments

Assignees

@Mcastres Mcastres

Labels

flag: don't merge This PR should not be merged at the moment pr: chore source: repo PRs/issues not targeting a specific documentation but rather affecting the whole repo target: v4 Documentation PRs/issues targeting content from docs.strapi.io (main branch).

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Mcastres @pwizla @derrickmehaffy