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

fix(clients): highlight and snippet results e2e #3567

Merged
merged 15 commits into from
Aug 21, 2024

Conversation

shortcuts
Copy link
Member

@shortcuts shortcuts commented Aug 20, 2024

🧭 What and Why

🎟 JIRA Ticket: https://algolia.atlassian.net/browse/DI-2767 https://algolia.atlassian.net/browse/DI-2776 https://algolia.atlassian.net/browse/DI-2779

Changes included:

reported in algolia/algoliasearch-client-kotlin#423, should close algolia/algoliasearch-client-kotlin#423

also enables e2e for kotlin, can't remember why it was missing but... it was, and highlights some problems

@shortcuts shortcuts self-assigned this Aug 20, 2024
@algolia-bot
Copy link
Collaborator

algolia-bot commented Aug 20, 2024

✔️ Code generated!

Name Link
🪓 Triggered by 1d2aed1b1348f1a6ab0776556e740cba4cad3816
🍃 Generated commit 4130f84684c2feec52babea8abf796f0529a47d1
🌲 Generated branch generated/chore/search-e2e-highlight-snippets
📊 Benchmark results

Benchmarks performed on the method using a mock server, the results might not reflect the real-world performance.

Language Rate
javascript 1360
php 1333
csharp 994
java 888
ruby 821
python 735
swift 709
go 536
kotlin 459

Copy link

github-actions bot commented Aug 20, 2024

@shortcuts shortcuts force-pushed the chore/search-e2e-highlight-snippets branch from cdf5e01 to 434ef86 Compare August 20, 2024 21:29
@millotp millotp marked this pull request as ready for review August 21, 2024 14:15
@millotp millotp requested a review from a team as a code owner August 21, 2024 14:15
Copy link
Collaborator

@millotp millotp left a comment

Choose a reason for hiding this comment

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

long time coming

@shortcuts
Copy link
Member Author

awesome thanks a lot for the help @millotp 😍

@shortcuts shortcuts merged commit 4cbbce5 into main Aug 21, 2024
21 checks passed
@shortcuts shortcuts deleted the chore/search-e2e-highlight-snippets branch August 21, 2024 14:58
algolia-bot added a commit that referenced this pull request Aug 21, 2024
…skip ci]

Co-authored-by: Clément Vannicatte <vannicattec@gmail.com>
Co-authored-by: Pierre Millot <pierre.millot@algolia.com>
algolia-bot added a commit to algolia/algoliasearch-client-csharp that referenced this pull request Aug 21, 2024
algolia/api-clients-automation#3567

Co-authored-by: algolia-bot <accounts+algolia-api-client-bot@algolia.com>
Co-authored-by: Clément Vannicatte <vannicattec@gmail.com>
Co-authored-by: Pierre Millot <pierre.millot@algolia.com>
algolia-bot added a commit to algolia/algoliasearch-client-go that referenced this pull request Aug 21, 2024
algolia/api-clients-automation#3567

Co-authored-by: algolia-bot <accounts+algolia-api-client-bot@algolia.com>
Co-authored-by: Clément Vannicatte <vannicattec@gmail.com>
Co-authored-by: Pierre Millot <pierre.millot@algolia.com>
algolia-bot added a commit to algolia/algoliasearch-client-java that referenced this pull request Aug 21, 2024
algolia/api-clients-automation#3567

Co-authored-by: algolia-bot <accounts+algolia-api-client-bot@algolia.com>
Co-authored-by: Clément Vannicatte <vannicattec@gmail.com>
Co-authored-by: Pierre Millot <pierre.millot@algolia.com>
algolia-bot added a commit to algolia/algoliasearch-client-javascript that referenced this pull request Aug 21, 2024
algolia/api-clients-automation#3567

Co-authored-by: algolia-bot <accounts+algolia-api-client-bot@algolia.com>
Co-authored-by: Clément Vannicatte <vannicattec@gmail.com>
Co-authored-by: Pierre Millot <pierre.millot@algolia.com>
algolia-bot added a commit to algolia/algoliasearch-client-kotlin that referenced this pull request Aug 21, 2024
algolia/api-clients-automation#3567

Co-authored-by: algolia-bot <accounts+algolia-api-client-bot@algolia.com>
Co-authored-by: Clément Vannicatte <vannicattec@gmail.com>
Co-authored-by: Pierre Millot <pierre.millot@algolia.com>
algolia-bot added a commit to algolia/algoliasearch-client-php that referenced this pull request Aug 21, 2024
algolia/api-clients-automation#3567

Co-authored-by: algolia-bot <accounts+algolia-api-client-bot@algolia.com>
Co-authored-by: Clément Vannicatte <vannicattec@gmail.com>
Co-authored-by: Pierre Millot <pierre.millot@algolia.com>
algolia-bot added a commit to algolia/algoliasearch-client-python that referenced this pull request Aug 21, 2024
algolia/api-clients-automation#3567

Co-authored-by: algolia-bot <accounts+algolia-api-client-bot@algolia.com>
Co-authored-by: Clément Vannicatte <vannicattec@gmail.com>
Co-authored-by: Pierre Millot <pierre.millot@algolia.com>
algolia-bot added a commit to algolia/algoliasearch-client-ruby that referenced this pull request Aug 21, 2024
algolia/api-clients-automation#3567

Co-authored-by: algolia-bot <accounts+algolia-api-client-bot@algolia.com>
Co-authored-by: Clément Vannicatte <vannicattec@gmail.com>
Co-authored-by: Pierre Millot <pierre.millot@algolia.com>
algolia-bot added a commit to algolia/algoliasearch-client-scala that referenced this pull request Aug 21, 2024
algolia/api-clients-automation#3567

Co-authored-by: algolia-bot <accounts+algolia-api-client-bot@algolia.com>
Co-authored-by: Clément Vannicatte <vannicattec@gmail.com>
Co-authored-by: Pierre Millot <pierre.millot@algolia.com>
algolia-bot added a commit to algolia/algoliasearch-client-swift that referenced this pull request Aug 21, 2024
algolia/api-clients-automation#3567

Co-authored-by: algolia-bot <accounts+algolia-api-client-bot@algolia.com>
Co-authored-by: Clément Vannicatte <vannicattec@gmail.com>
Co-authored-by: Pierre Millot <pierre.millot@algolia.com>
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.

Failed to parse _highlightResult
3 participants