Skip to content

Add correct deserialization class to listInAppPurchasesV2PricePoints - #39

Open
srowan wants to merge 3 commits into
cantie:mainfrom
srowan:patch-1
Open

Add correct deserialization class to listInAppPurchasesV2PricePoints#39
srowan wants to merge 3 commits into
cantie:mainfrom
srowan:patch-1

Conversation

@srowan

@srowan srowan commented Feb 5, 2025

Copy link
Copy Markdown

Small bug fix. This was using the wrong class (csv?)

@srowan

srowan commented Feb 5, 2025

Copy link
Copy Markdown
Author

Sorry, not sure what's going on with the whitespace differences 😓

@cantie

cantie commented Feb 8, 2025

Copy link
Copy Markdown
Owner

I've look at this issue, since Resource classes are generated from here:
https://developer.apple.com/sample-code/app-store-connect/app-store-connect-openapi-specification.zip so in this case I believe it was unproperly handle.
In some API if response with 200 HTTP status code, the content object is can be csv or another resource object.
image

I've planned to open the parse openapi.json project also, so maybe you can help me check this out?
For now I need time to review other APIs with the same situation to make sure we can use Resource object instead of csv.

Best Regards,

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