Skip to content

Conversation

kai687
Copy link
Contributor

@kai687 kai687 commented Jun 3, 2025

🧭 What and Why

Add title properties to the oneOf options for highlightResult and snippetResult in the search response.

Without them, our work-in-progress Mintlify docs just show 'option 1', 'option 2' etc. in the oneOf selector.

🎟 JIRA Ticket:

Changes included:

  • List changes

🧪 Test

@algolia-bot
Copy link
Collaborator

algolia-bot commented Jun 3, 2025

💥 Breaking changes detected !!

Either this PR or a previous PR not released yet introduced breaking changes, be careful when merging.
You can find the details in the 'client java@17' CI job.

✔️ Code generated!

Name Link
🪓 Triggered by f5b8c716fc9c7458da87809143c8702de1492d56
🍃 Generated commit a61ff70c8e955322d25d73b3804002767c964cb8
🌲 Generated branch generated/fix/titles-oneof
📊 Benchmark results

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

Language Req/s
go 1780
javascript 1661
php 1520
csharp 1285
java 1091
python 1062
ruby 811
swift 773
scala 23

@kai687 kai687 marked this pull request as ready for review June 3, 2025 11:48
@kai687 kai687 requested a review from a team as a code owner June 3, 2025 11:48
@kai687 kai687 requested review from morganleroi and shortcuts June 3, 2025 11:48
@shortcuts
Copy link
Member

Looks great! Is this required for every oneOf? We could make it a custom eslint rule if necessary

@shortcuts shortcuts merged commit f91b4e4 into main Jun 3, 2025
53 of 55 checks passed
@shortcuts shortcuts deleted the fix/titles-oneof branch June 3, 2025 12:20
algolia-bot added a commit that referenced this pull request Jun 3, 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.

3 participants