Skip to content
This repository was archived by the owner on Jan 23, 2026. It is now read-only.

json_util: Add support for loading from a string - #530

Merged
mattkjames7 merged 3 commits into
memgraph:mainfrom
pmundt:json-load-from-str
Dec 15, 2025
Merged

json_util: Add support for loading from a string#530
mattkjames7 merged 3 commits into
memgraph:mainfrom
pmundt:json-load-from-str

Conversation

@pmundt

@pmundt pmundt commented Dec 12, 2024

Copy link
Copy Markdown
Contributor

Description

Extended the json_util query module to support loading from a passed in JSON string. This can be helpful in cases where a client wishes to e.g. create node properties dynamically from an existing JSON representation via client APIs.

Pull request type

  • Bugfix
  • Algorithm/Module
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

######################################

Reviewer checklist (the reviewer checks this part)

Module/Algorithm

  • Core algorithm/module implementation
  • Query module implementation
  • Tests provided (unit / e2e)
  • Code documentation
  • README short description

Documentation checklist

  • Add the documentation label tag
  • Add the bug / feature label tag
  • Add the milestone for which this feature is intended
    • If not known, set for a later milestone
  • Write a release note, including added/changed clauses
    • Added load_from_str() function to the json_util query module in order to allow loading JSON directly from a string. #530
  • Link the documentation PR here
  • Tag someone from docs team in the comments

Extend the json_util query module to support loading from a passed in JSON
string. This can be helpful in cases where a client wishes to e.g. create
node properties dynamically from an existing JSON representation via client
APIs.
@sonarqubecloud

Copy link
Copy Markdown

@CLAassistant

CLAassistant commented Mar 5, 2025

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@mattkjames7 mattkjames7 self-assigned this Dec 15, 2025
@mattkjames7 mattkjames7 added feature feature Docs needed Docs needed labels Dec 15, 2025
@mattkjames7 mattkjames7 added this to the mage-v3.8.0 milestone Dec 15, 2025
@mattkjames7
mattkjames7 added this pull request to the merge queue Dec 15, 2025
github-merge-queue Bot pushed a commit that referenced this pull request Dec 15, 2025
### Description

Extended the json_util query module to support loading from a passed in
JSON string. This can be helpful in cases where a client wishes to e.g.
create node properties dynamically from an existing JSON representation
via client APIs.

### Pull request type

- [ ] Bugfix
- [x] Algorithm/Module
- [x] Feature
- [ ] Code style update (formatting, renaming)
- [ ] Refactoring (no functional changes, no api changes)
- [ ] Build related changes
- [ ] Documentation content changes
- [ ] Other (please describe):

######################################

### Reviewer checklist (the reviewer checks this part)
#### Module/Algorithm
- [ ] Core algorithm/module implementation
- [ ] [Query
module](https://memgraph.com/docs/memgraph/reference-guide/query-modules)
implementation
- [ ] Tests provided (unit / e2e)
- [ ] Code documentation
- [ ] README short description


### Documentation checklist
- [ ] Add the documentation label tag
- [ ] Add the bug / feature label tag
- [ ] Add the milestone for which this feature is intended
    - If not known, set for a later milestone
- [ ] Write a release note, including added/changed clauses
- Added `load_from_str()` function to the `json_util` query module in
order to allow loading JSON directly from a string.
[#530](#530)
- [ ] Link the documentation PR here
    - memgraph/documentation#1491
- [ ] Tag someone from docs team in the comments

Co-authored-by: Dr Matt James <mattkjames7@gmail.com>
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Dec 15, 2025
@mattkjames7
mattkjames7 added this pull request to the merge queue Dec 15, 2025
@sonarqubecloud

Copy link
Copy Markdown

@mattkjames7
mattkjames7 removed this pull request from the merge queue due to a manual request Dec 15, 2025
@mattkjames7
mattkjames7 merged commit c728651 into memgraph:main Dec 15, 2025
6 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Docs needed Docs needed feature feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants