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.

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

update algolia token #26

Merged
merged 1 commit into from
Oct 10, 2023
Merged

update algolia token #26

merged 1 commit into from
Oct 10, 2023

Conversation

matthieu-rolland
Copy link
Contributor

@matthieu-rolland matthieu-rolland commented Oct 10, 2023 •
edited
Loading

Questions Answers
Description? update algolia token
Fixed ticket? Fixes #{issue number here} if there is a related issue

@matthieu-rolland matthieu-rolland changed the title refresh algolia token update algolia token Oct 10, 2023
@matthieu-rolland matthieu-rolland marked this pull request as ready for review October 10, 2023 08:45
thomasnares
thomasnares approved these changes Oct 10, 2023
View reviewed changes
Copy link

@thomasnares thomasnares left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Tested locally with old token: works 🟢
Tested locally with bad token: no results 🔴
Tested locally with new token: works 🟢

That's good for me 👍

@matthieu-rolland
Copy link
Contributor Author

thank you guys !

@matthieu-rolland matthieu-rolland merged commit cafa0e2 into PrestaShop:main Oct 10, 2023
1 check passed
@matks
Copy link
Contributor

matks commented Oct 10, 2023

@matthieu-rolland as the documentation states https://github.com/PrestaShop/devdocs-site/blob/main/README.md

Theme updates do not trigger a deployment, consequently following a theme update, the docs website must be deployed manually.

So you must do a manual deploy

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jolelievre jolelievre jolelievre approved these changes

@thomasnares thomasnares thomasnares approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@matthieu-rolland @matks @thomasnares @jolelievre