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(specs): update composition specs according to upcoming API changes #4234

Merged
merged 9 commits into from
Dec 18, 2024

Conversation

e-krebs
Copy link
Contributor

@e-krebs e-krebs commented Dec 12, 2024

🧭 What and Why

Update composition specs according to upcoming API changes
https://algolia.atlassian.net/wiki/spaces/CMP/pages/5556273184/DD+-+Composition+API+updates

🎟 JIRA Ticket: EMERCH-1797

@e-krebs e-krebs self-assigned this Dec 12, 2024
@algolia-bot
Copy link
Collaborator

algolia-bot commented Dec 12, 2024

✔️ Code generated!

Name Link
🪓 Triggered by 8adb8ce21901b1cff5c00a43f2c99fe93b38808e
🍃 Generated commit feeaf994abaeee8aa7c26e005d992a54117c14c2
🌲 Generated branch generated/feat/composition-schema-update
📊 Benchmark results

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

Language Req/s
go 1644
javascript 1619
php 1440
csharp 1388
python 1108
java 1101
ruby 869
swift 738

@e-krebs e-krebs changed the title feat: update composition specs according to upcoming API changes feat(specs): update composition specs according to upcoming API changes Dec 12, 2024
@e-krebs e-krebs force-pushed the feat/composition-schema-update branch 2 times, most recently from 4ede08f to 5febb5c Compare December 12, 2024 14:19
@e-krebs e-krebs marked this pull request as ready for review December 17, 2024 10:22
@e-krebs e-krebs requested a review from a team as a code owner December 17, 2024 10:22
@e-krebs
Copy link
Contributor Author

e-krebs commented Dec 17, 2024

@shortcuts I previously mis-implemented the search response, it is now fixed (cf. doc linked in description for the correct shape)

@e-krebs e-krebs requested a review from shortcuts December 17, 2024 14:14
Copy link
Member

@shortcuts shortcuts left a comment

Choose a reason for hiding this comment

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

thanks!

Copy link
Member

Choose a reason for hiding this comment

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

would be great to have more concrete e2e so we can assert those changes, can be done later though

@shortcuts shortcuts enabled auto-merge (squash) December 18, 2024 10:17
@shortcuts shortcuts merged commit 448aad7 into main Dec 18, 2024
28 checks passed
@shortcuts shortcuts deleted the feat/composition-schema-update branch December 18, 2024 10:30
algolia-bot added a commit that referenced this pull request Dec 18, 2024
…es (#4234) (generated) [skip ci]

Co-authored-by: Emmanuel Krebs <e-krebs@users.noreply.github.com>
Co-authored-by: shortcuts <vannicattec@gmail.com>
algolia-bot added a commit to algolia/algoliasearch-client-javascript that referenced this pull request Dec 18, 2024
…es (generated)

algolia/api-clients-automation#4234

Co-authored-by: algolia-bot <accounts+algolia-api-client-bot@algolia.com>
Co-authored-by: Emmanuel Krebs <e-krebs@users.noreply.github.com>
Co-authored-by: shortcuts <vannicattec@gmail.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.

4 participants