Skip to content

Conversation

@kai687
Copy link
Contributor

@kai687 kai687 commented Nov 12, 2025

🧭 What and Why

Fix a small issue for the "guide" snippet for deleting multiple indices.
When looping over replica indices, the slice length should be of length replicaIndices, not primaryIndices.

(From documentation feedback)

🎟 JIRA Ticket: DOC-1710

Changes included:

  • List changes

🧪 Test

@algolia-bot
Copy link
Collaborator

algolia-bot commented Nov 12, 2025

✔️ Code generated!

Name Link
🪓 Triggered by 479f7e03f814cb3fd6537181f5326cc09a354777
🍃 Generated commit c2250cd4b55f77f03f288dd0cf7b31fe28da9b31
🌲 Generated branch generated/fix/guide/deleteMultipleIndices
📊 Benchmark results

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

Language Req/s
go 2296

@kai687 kai687 changed the title fix(guides): error in deleteMultipleIndices guide guides: fix error in deleteMultipleIndices template for Go Nov 12, 2025
@kai687 kai687 marked this pull request as ready for review November 12, 2025 09:29
@kai687 kai687 requested a review from a team as a code owner November 12, 2025 09:29
@kai687 kai687 requested review from Fluf22 and morganleroi November 12, 2025 09:29
Copy link
Contributor

@millotp millotp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks !

@millotp millotp merged commit 12b1e76 into main Nov 12, 2025
13 of 15 checks passed
@millotp millotp deleted the fix/guide/deleteMultipleIndices branch November 12, 2025 09:38
algolia-bot added a commit that referenced this pull request Nov 12, 2025
…enerated) [skip ci]

Co-authored-by: Kai Welke <kai.welke@algolia.com>
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