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

feat: Upgrade algoliasearch v5 #219

Merged
merged 5 commits into from
Jan 29, 2025
Merged

feat: Upgrade algoliasearch v5 #219

merged 5 commits into from
Jan 29, 2025

Conversation

htuzel
Copy link
Collaborator

@htuzel htuzel commented Jan 14, 2025

What is changed

  • Updated algoliasearch package from version 4.24.0 to 5.19.0.
  • Updated instantsearch.js and instantsearch.css to versions 4.75.7 and 8.5.1 respectively.
  • Refactored file paths in copy-files-from-to.json to reflect new library structure.

- Updated `algoliasearch` package from version 4.24.0 to 5.19.0.
- Updated `instantsearch.js` and `instantsearch.css` to versions 4.75.7 and 8.5.1 respectively.
- Refactored file paths in `copy-files-from-to.json` to reflect new library structure.
- Changed import statement for `algoliasearch` to destructured format.
- Updated test setup to use `client.searchSingleIndex` instead of `index.search` for improved clarity and alignment with the latest Algolia API.
- Adjusted search parameters to utilize the environment variable for index name.

These changes enhance the maintainability and readability of the test code while ensuring compatibility with the updated Algolia library.
@htuzel htuzel self-assigned this Jan 14, 2025
@htuzel htuzel requested a review from sbellone January 17, 2025 13:21
Copy link
Collaborator

@sbellone sbellone left a comment

Choose a reason for hiding this comment

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

Seems to work fine.
Just merge develop to have the latest versions of the frontend library, otherwise it looks ok.

package.json Outdated
@@ -36,7 +36,7 @@
"@algolia/autocomplete-theme-classic": "1.12.1",
Copy link
Collaborator

@sbellone sbellone Jan 24, 2025

Choose a reason for hiding this comment

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

It's a good occasion to upgrade Autocomplete too.
Actually I did it in #180, so if you merge develop it should be updated to the latest version.
Sorry for the conflicts, normally you can just take the version from develop

htuzel and others added 2 commits January 27, 2025 13:54
…derScripts.isml

Co-authored-by: Sylvain Bellone <sylvain.bellone@algolia.com>
@htuzel htuzel requested a review from sbellone January 29, 2025 11:15
@htuzel htuzel merged commit 86058ef into develop Jan 29, 2025
5 checks passed
@htuzel htuzel deleted the feat/SFCC-292 branch January 29, 2025 11:15
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