Skip to content

Conversation

@millotp
Copy link
Collaborator

@millotp millotp commented Jul 4, 2025

🧭 What and Why

🎟 JIRA Ticket: CR-8889

Time ranges for rule can be one sided, we can't express that in openapi without using a one of, so we just make them both optional.

@millotp millotp requested a review from a team as a code owner July 4, 2025 08:40
@millotp millotp requested review from Fluf22 and morganleroi July 4, 2025 08:40
@millotp millotp self-assigned this Jul 4, 2025
@algolia-bot
Copy link
Collaborator

algolia-bot commented Jul 4, 2025

✔️ Code generated!

Name Link
🪓 Triggered by 43b30a03accfaaa59beca37652e3520cf7b4b07a
🍃 Generated commit 8ba91da8aaff5faf1a79628699ffd6d0a2f7ae0e
🌲 Generated branch generated/fix/timerange
📊 Benchmark results

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

Language Req/s
go 1784
javascript 1629
php 1482
csharp 1351
java 1077
python 1070
ruby 803
swift 352
scala 23

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.

this is a bc for go no?

@millotp
Copy link
Collaborator Author

millotp commented Jul 4, 2025

this is a bc for go no?

indeed, and scala, but the spec was not correct so it's still a fix right ?

@shortcuts
Copy link
Member

this is a bc for go no?

indeed, and scala, but the spec was not correct so it's still a fix right ?

yup, just makes me think that we should have a way to properly reflect those in changelogs

@millotp millotp enabled auto-merge (squash) July 4, 2025 10:59
@millotp millotp merged commit 754efce into main Jul 4, 2025
29 checks passed
@millotp millotp deleted the fix/timerange branch July 4, 2025 11:09
algolia-bot added a commit that referenced this pull request Jul 4, 2025
algolia-bot added a commit to algolia/algoliasearch-client-csharp that referenced this pull request Jul 4, 2025
algolia-bot added a commit to algolia/algoliasearch-client-dart that referenced this pull request Jul 4, 2025
algolia-bot added a commit to algolia/algoliasearch-client-go that referenced this pull request Jul 4, 2025
algolia-bot added a commit to algolia/algoliasearch-client-java that referenced this pull request Jul 4, 2025
algolia-bot added a commit to algolia/algoliasearch-client-javascript that referenced this pull request Jul 4, 2025
algolia-bot added a commit to algolia/algoliasearch-client-kotlin that referenced this pull request Jul 4, 2025
algolia-bot added a commit to algolia/algoliasearch-client-php that referenced this pull request Jul 4, 2025
algolia-bot added a commit to algolia/algoliasearch-client-python that referenced this pull request Jul 4, 2025
algolia-bot added a commit to algolia/algoliasearch-client-ruby that referenced this pull request Jul 4, 2025
algolia-bot added a commit to algolia/algoliasearch-client-scala that referenced this pull request Jul 4, 2025
algolia-bot added a commit to algolia/algoliasearch-client-swift that referenced this pull request Jul 4, 2025
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