Skip to content

DEV: fix bug on indexing page #1735

New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 2 commits into from
Jun 23, 2025
Merged

DEV: fix bug on indexing page #1735

merged 2 commits into from
Jun 23, 2025

Conversation

@dwdougherty
Copy link
Collaborator

@dwdougherty dwdougherty commented Jun 23, 2025 •
edited by atlassian bot
Loading

DOC-5357

Change:

"You cannot index JSON objects. If the JSONPath expression returns an object, it will be ignored."

to:

"You cannot index JSON objects. If the JSONPath expression returns an object, an indexing error will be returned."

@dwdougherty dwdougherty requested a review from a team June 23, 2025 14:14
@dwdougherty dwdougherty self-assigned this Jun 23, 2025
@dwdougherty dwdougherty added bug Something isn't working dev labels Jun 23, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Jun 23, 2025

Staging links:
https://redis.io/docs/staging/DOC-5357/develop/interact/search-and-query/indexing/

andy-stark-redis
andy-stark-redis approved these changes Jun 23, 2025
View reviewed changes
Copy link
Contributor

@andy-stark-redis andy-stark-redis left a comment

Choose a reason for hiding this comment

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

LGTM

@dwdougherty
Copy link
Collaborator Author

dwdougherty commented Jun 23, 2025

Thanks, @andy-stark-redis.

@dwdougherty dwdougherty merged commit 6b52720 into main Jun 23, 2025
5 checks passed
@dwdougherty dwdougherty deleted the DOC-5357 branch June 23, 2025 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@andy-stark-redis andy-stark-redis andy-stark-redis approved these changes

Assignees

@dwdougherty dwdougherty

Labels

bug Something isn't working dev

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@dwdougherty @andy-stark-redis