Skip to content

DOC-4758 async JSON doc examples #3335

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.

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

andy-stark-redis
Copy link
Contributor

@andy-stark-redis andy-stark-redis commented Jun 25, 2025

Make sure that:

  • You have read the contribution guidelines.
  • You applied code formatting rules using the mvn formatter:format target. Don’t submit any formatting related changes.
  • You submit test cases (unit or integration tests) that back your changes.

DOC-4758

Lettuce async examples for the JSON landing page and the Path page.

Note: I've used arrInsert() instead of arrAppend() on line 195 because the arrAppend() method doesn't seem to work when the array is at the root path. The implementation seems to assume that the path argument is optional when the path is the root, as it says in the command doc page. However, I think the docs might be wrong, because this doesn't work in the CLI either. I'll look into this.

@andy-stark-redis andy-stark-redis marked this pull request as ready for review June 25, 2025 13:38
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.

1 participant