Skip to content

Comments

Documentation: Add details on jsonpath filter#858

Merged
oshadmi merged 3 commits intomasterfrom
omer_jsonpath_filter_details-docs
Nov 9, 2022
Merged

Documentation: Add details on jsonpath filter#858
oshadmi merged 3 commits intomasterfrom
omer_jsonpath_filter_details-docs

Conversation

@oshadmi
Copy link
Collaborator

@oshadmi oshadmi commented Nov 9, 2022

No description provided.

nermiller
nermiller previously approved these changes Nov 9, 2022
Copy link
Contributor

@nermiller nermiller left a comment

Choose a reason for hiding this comment

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

Just a few minor edits, @oshadmi. Thank you!

Co-authored-by: Nermina Miller <102551568+nermiller@users.noreply.github.com>
@oshadmi oshadmi requested a review from nermiller November 9, 2022 19:51
Copy link
Contributor

@nermiller nermiller left a comment

Choose a reason for hiding this comment

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

LGTM!

@oshadmi oshadmi merged commit f87c93d into master Nov 9, 2022
@oshadmi oshadmi deleted the omer_jsonpath_filter_details-docs branch November 9, 2022 20:06
rafie pushed a commit that referenced this pull request Nov 13, 2022
* Add details on jsonpath filter

* fix local path and some edits

* Apply suggestions from review by Nermina

Co-authored-by: Nermina Miller <102551568+nermiller@users.noreply.github.com>

Co-authored-by: Nermina Miller <102551568+nermiller@users.noreply.github.com>
(cherry picked from commit f87c93d)
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.

2 participants