-
Notifications
You must be signed in to change notification settings - Fork 22
fix(specs): Add customSearchParameters to Variant schema, and add direction query param for GET abtests #5130
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
Conversation
… param for GET abtests
✔️ Code generated!
📊 Benchmark resultsBenchmarks performed on the method using a mock server, the results might not reflect the real-world performance.
|
Hey @shortcuts, would it be possible re-release the alpha client after this gets merged? Thanks in advance for your help / review! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hey sorry I was already OOO at that time, I'd recommend you to set your team as codeowners here https://github.com/algolia/api-clients-automation/blob/main/CODEOWNERS so that you can go ahead with the specs change and potentially release the clients using this GitHub action https://github.com/algolia/api-clients-automation/actions/workflows/scheduled-release.yml
…ection query param for GET abtests (#5130) (generated) [skip ci] Co-authored-by: Leonardo Gavaudan <leonardogavaudan@gmail.com> Co-authored-by: Clément Vannicatte <vannicattec@gmail.com>
…ection query param for GET abtests (generated) algolia/api-clients-automation#5130 Co-authored-by: algolia-bot <accounts+algolia-api-client-bot@algolia.com> Co-authored-by: Leonardo Gavaudan <leonardogavaudan@gmail.com> Co-authored-by: Clément Vannicatte <vannicattec@gmail.com>
Two changes:
direction
forGET /abtests
customSearchParameters
in theVariant
schema which is then used in theGET /abtests/
response🧭 What and Why
🎟 JIRA Ticket:
Changes included:
🧪 Test