Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(specs): add virtual index settings and update renderingContent #3254

Merged
merged 1 commit into from
Jun 25, 2024

Conversation

shortcuts
Copy link
Member

🧭 What and Why

🎟 JIRA Ticket: https://algolia.atlassian.net/browse/DI-2514

Changes included:

port latest changes from the js repository

@shortcuts shortcuts requested a review from a team as a code owner June 25, 2024 16:08
@shortcuts shortcuts requested review from Fluf22 and millotp June 25, 2024 16:08
@shortcuts shortcuts self-assigned this Jun 25, 2024
@shortcuts shortcuts enabled auto-merge (squash) June 25, 2024 16:08
@algolia-bot
Copy link
Collaborator

algolia-bot commented Jun 25, 2024

✗ The generated branch has been deleted.

If the PR has been merged, you can check the generated code on the main branch.
You can still access the code generated on main via this commit.

Copy link
Contributor

@Fluf22 Fluf22 left a comment

Choose a reason for hiding this comment

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

🆗

Copy link

@github-actions github-actions bot temporarily deployed to pull request June 25, 2024 16:23 Inactive
@shortcuts shortcuts merged commit 0a62779 into main Jun 25, 2024
21 checks passed
@shortcuts shortcuts deleted the feat/specs-search branch June 25, 2024 16:23
Comment on lines +67 to +69
virtual:
type: boolean
description: Only present if the index is a [virtual replica](https://www.algolia.com/doc/guides/managing-results/refine-results/sorting/how-to/sort-an-index-alphabetically/#virtual-replicas).
Copy link
Contributor

Choose a reason for hiding this comment

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

Where is this supposed to show up? I can't use it with setSettings and I don't see it in the response for a virtual replica with getSettings? Is this used somewhere?

Copy link
Member Author

Choose a reason for hiding this comment

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

good point, fixed in #3255, it's actually only returned when fetching the index list

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants