Skip to content

Conversation

cdhawke
Copy link
Contributor

@cdhawke cdhawke commented Jul 15, 2025

🧭 What and Why

image

The value is missing from the v3 definitions, but is returned from the API

Changes included:

Add the property to the metadata metric definition

@cdhawke cdhawke self-assigned this Jul 15, 2025
@cdhawke cdhawke requested a review from a team as a code owner July 15, 2025 17:31
@cdhawke cdhawke requested review from millotp and shortcuts July 15, 2025 17:31
@algolia-bot
Copy link
Collaborator

algolia-bot commented Jul 15, 2025

✔️ Code generated!

Name Link
🪓 Triggered by f826986c9fa5cae8f8fec8973a52c329e624651a
🍃 Generated commit e6414d2c92381c13f7d8127e8edabcfb7a5595bf
🌲 Generated branch generated/fix/abtests/metadata-mean
📊 Benchmark results

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

Language Req/s
go 1748
javascript 1683
php 1519
csharp 1370
java 1117
python 1070
ruby 649
swift 606
scala 23

@shortcuts shortcuts merged commit d059997 into main Jul 16, 2025
29 checks passed
@shortcuts shortcuts deleted the fix/abtests/metadata-mean branch July 16, 2025 06:36
algolia-bot added a commit that referenced this pull request Jul 16, 2025
algolia-bot added a commit to algolia/algoliasearch-client-javascript that referenced this pull request Jul 16, 2025
@cdhawke
Copy link
Contributor Author

cdhawke commented Jul 17, 2025

@shortcuts what's the cadence for publishing this to npm? is this alpha version a manual process?

@shortcuts
Copy link
Member

@shortcuts what's the cadence for publishing this to npm? is this alpha version a manual process?

whenever you feel like you need it feel free to ask and we can trigger a release process! we don't release automatically to prevent spamming new versions

@cdhawke
Copy link
Contributor Author

cdhawke commented Jul 17, 2025

Could you release this then? it's blocking https://github.com/algolia/AlgoliaWeb/pull/25052 (I could add type overrides but wanted to avoid the tech debt and just fix it here)

@shortcuts
Copy link
Member

sure #5127

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.

5 participants