Skip to content

Conversation

@PierreZ
Copy link
Contributor

@PierreZ PierreZ commented Mar 12, 2025

Describe your PR

Summarize your changes here : explain what, how, and why. Be as explicict as you can on why your changes are needed and avoid implicit reasoning.
This pull request includes updates to the Materia KV documentation to introduce and explain new JSON functions. The changes include a new changelog entry, updates to the list of supported commands, and detailed examples and limitations of the JSON functions.

Documentation updates:

Examples and usage:

  • content/doc/addons/materia-kv.md: Added a section with detailed examples of using JSON functions with redis-cli, including setting, getting, and deleting JSON values, as well as handling multiple paths and recursive searches.

Limitations:

  • content/doc/addons/materia-kv.md: Documented the current limitations of the JSON functions, such as restrictions on creating new fields in existing documents and nested path creation.

Checklist

Reviewers

Who should review these changes? @CleverCloud/reviewers

@PierreZ PierreZ requested a review from davlgd March 12, 2025 09:21
@github-actions
Copy link

github-actions bot commented Mar 12, 2025

✅ Review app deployed

Check how your changes look like!

ℹ️ Name 🔗 Infos & links
🌱 Latest deployed commit 7a02ef2
👁️ Preview https://documentation-PR-561.cleverapps.io/developers

Co-Authored-By: Pierre Zemb <contact@pierrezemb.fr>
Co-Authored-By: RaspY <161023878+raspy737@users.noreply.github.com>
davlgd and others added 2 commits March 13, 2025 14:33
Co-Authored-By: Pierre Zemb <contact@pierrezemb.fr>
Co-Authored-By: RaspY <161023878+raspy737@users.noreply.github.com>
Copy link
Collaborator

@davlgd davlgd left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for this!

@davlgd davlgd merged commit bba4863 into main Mar 13, 2025
4 of 7 checks passed
@davlgd davlgd deleted the kv-json-release branch March 13, 2025 13:42
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