Skip to content
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

Add recordPath option to JsonDecoder #384

Merged
merged 3 commits into from
Aug 27, 2021
Merged

Add recordPath option to JsonDecoder #384

merged 3 commits into from
Aug 27, 2021

Conversation

fsteeg
Copy link
Member

@fsteeg fsteeg commented Aug 25, 2021

Will resolve #382

Instead of using the entire JSON as a single record, provide a
JSON path to query the JSON for the records to process, e.g.
`$.data` to process every entry in a `data` array as a record.
@dr0i
Copy link
Member

dr0i commented Aug 26, 2021

Added flux annotations to also be able to update flux-commands documentation, see metafacture/metafacture-documentation#14.

Copy link
Member

@dr0i dr0i left a comment

Choose a reason for hiding this comment

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

Added flux annotations.

@dr0i dr0i assigned fsteeg and unassigned dr0i Aug 26, 2021
@dr0i
Copy link
Member

dr0i commented Aug 26, 2021

Cannot assign @fsteeg as reviewer. Note that I added a new commit, please review this.

fsteeg added a commit to metafacture/metafacture-documentation that referenced this pull request Aug 27, 2021
@fsteeg fsteeg merged commit fdf1832 into master Aug 27, 2021
@dr0i dr0i deleted the 382-jsonApi branch November 25, 2021 08:58
blackwinter pushed a commit that referenced this pull request Dec 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add or enhance a function to extract JSON-Records from an JSON-API
2 participants