Skip to content

Conversation

leonardogavaudan
Copy link
Contributor

Two changes:

  • added query parameter direction for GET /abtests
  • added customSearchParameters in the Variant schema which is then used in the GET /abtests/ response

🧭 What and Why

🎟 JIRA Ticket:

Changes included:

  • List changes

🧪 Test

@leonardogavaudan leonardogavaudan requested a review from a team as a code owner July 18, 2025 14:22
@algolia-bot
Copy link
Collaborator

algolia-bot commented Jul 18, 2025

✔️ Code generated!

Name Link
🪓 Triggered by 5346586db21e62ff8fcb1dd830f7cfbb52a617e6
🍃 Generated commit 666780739bb871f744b51d5cd6579dae8d2aa1e2
🌲 Generated branch generated/feat/abtests/api-ref-updates
📊 Benchmark results

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

Language Req/s
go 1757
javascript 1622
php 1558
csharp 1451
python 1105
java 1079
ruby 803
swift 624
scala 23

@leonardogavaudan
Copy link
Contributor Author

Hey @shortcuts, would it be possible re-release the alpha client after this gets merged? Thanks in advance for your help / review!

@leonardogavaudan leonardogavaudan changed the title Add customSearchParameters to Variant schema, and add direction query param for GET abtests feat(abtest): Add customSearchParameters to Variant schema, and add direction query param for GET abtests Jul 18, 2025
@leonardogavaudan leonardogavaudan changed the title feat(abtest): Add customSearchParameters to Variant schema, and add direction query param for GET abtests fix(specs): Add customSearchParameters to Variant schema, and add direction query param for GET abtests Jul 18, 2025
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.

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

cc @cdhawke @leonardogavaudan

@shortcuts shortcuts enabled auto-merge (squash) July 21, 2025 08:54
@shortcuts shortcuts merged commit d1596a8 into main Jul 21, 2025
29 checks passed
@shortcuts shortcuts deleted the feat/abtests/api-ref-updates branch July 21, 2025 09:11
algolia-bot added a commit that referenced this pull request Jul 21, 2025
…ection query param for GET abtests (#5130) (generated) [skip ci]

Co-authored-by: Leonardo Gavaudan <[email protected]>
Co-authored-by: Clément Vannicatte <[email protected]>
algolia-bot added a commit to algolia/algoliasearch-client-javascript that referenced this pull request Jul 21, 2025
…ection query param for GET abtests (generated)

algolia/api-clients-automation#5130

Co-authored-by: algolia-bot <[email protected]>
Co-authored-by: Leonardo Gavaudan <[email protected]>
Co-authored-by: Clément Vannicatte <[email protected]>
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