Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

keyvaluetype_resultType#162

Open
ggibson-github wants to merge 1 commit intoJSONPath-Plus:main from
ggibson-github:keyvaluetype_resultType
Open

keyvaluetype_resultType #162
ggibson-github wants to merge 1 commit intoJSONPath-Plus:main from
ggibson-github:keyvaluetype_resultType

Conversation

@ggibson-github
Copy link

@ggibson-github ggibson-github commented Oct 17, 2021

PR description

Add resultType option to return a JSON object containing key, value, and type.

Add resultType option to return a JSON object containing key, value, and type.
Copy link
Collaborator

brettz9 commented Oct 21, 2021

Is there a reason you do not just use the all return type?

Copy link
Author

ggibson-github commented Oct 21, 2021 via email

Returns far more data than needed and did not include the data type. Key-Value pairs are a very common scenario and in some cases having the JavaScript data type is important as well. I made the change for my project and shared it with you. If it does not fit the model you have in mind just ignore it. From: Brett ***@***.***> Sent: Thursday, October 21, 2021 6:44 AM To: ***@***.***> Cc: ***@***.***>; ***@***.***> Subject: Re: [JSONPath-Plus/JSONPath] keyvaluetype_resultType (#162) Is there a reason you do not just use the all return type? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub<#162 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AMXISN2CV5PPKRL43RNEXLDUH7VHPANCNFSM5GEHM4EQ>. Triage notifications on the go with GitHub Mobile for iOS<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /