Skip to content

Add include query parameter to extract endpoint#30

Open
HaasCM wants to merge 1 commit intoGICodeWarrior:mainfrom
HaasCM:haascm/extend-post
Open

Add include query parameter to extract endpoint#30
HaasCM wants to merge 1 commit intoGICodeWarrior:mainfrom
HaasCM:haascm/extend-post

Conversation

@HaasCM
Copy link
Contributor

@HaasCM HaasCM commented Mar 21, 2026

Add a new query parameter to the extract endpoint to pull additional information from the version's catalog. The additional information may be passed in via a comma delimited list of jquery like strings. The additional information is returned to the user in an attributes struct in the JSON response.

@HaasCM HaasCM force-pushed the haascm/extend-post branch 2 times, most recently from 22c01d9 to 5d06bef Compare March 21, 2026 23:47
Add a new query parameter to the extract endpoint to pull additional information
from the version's catalog. The additional information may be passed in via a
comma delimited list of jquery like strings. The additional information is
returned to the user in an attributes struct in the JSON response.
@HaasCM HaasCM force-pushed the haascm/extend-post branch from 5d06bef to 4f050be Compare March 21, 2026 23:48
@HaasCM
Copy link
Contributor Author

HaasCM commented Mar 21, 2026

PR to address #29. Feel free to change up, close, or use as a starting point. I'm relatively new to rust, so there may be some mistakes.

To test:

curl -v --data-binary @stockpile.png "http://127.0.0.1:8080/extract?include=.ItemDynamicData"

or

curl -v --data-binary @stockpile.png "http://127.0.0.1:8080/extract?include=.ItemDynamicData.QuantityPerCrate"

it does support multiple includes such as:

curl -v --data-binary @stockpile.png "http://127.0.0.1:8080/extract?include=.ItemDynamicData.QuantityPerCrate,.ItemDynamicData.CrateRetrieveTime"

I elected for a jquery like label for the include query parameter.

Lemme know what you think.

@HaasCM
Copy link
Contributor Author

HaasCM commented Mar 21, 2026

I may have added way too many packages, I did run cargo fix --allow-dirty --allow-staged to verify that I am using all of those packages somewhere.

@GICodeWarrior
Copy link
Owner

Thanks for this. I'll review tomorrow or the day after. 👍🏻

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.

2 participants