recurly / recurly-client-python

Recurly Python Client
https://developers.recurly.com
MIT License
92 stars 78 forks source link

Return resume_at field when retrieving subscriptions #685

Closed enberg closed 7 months ago

enberg commented 7 months ago

Currently the client doesn't return the resume_at field when retrieving subscriptions.

The field exists and is returned from the API but the python client omits it.

douglasmiller commented 7 months ago

@enberg, could you please clarify which version of the python client you are using?

enberg commented 7 months ago

Hey @douglasmiller!

Currently on recurly = "==4.40.0"

douglasmiller commented 7 months ago

@enberg, I manually checked raw API responses (using curl) for subscriptions and I don't see a resume_at property being returned. I also looked through the code that renders the API response and there is no resume_at property being rendered.

The only place that I see resume_at documented anywhere is in our XML webhooks. Are you attempting to use the JSON python client to parse the XML payloads?

enberg commented 7 months ago

Probably a miss on our end, mixing up webhook xml response with the API, sorry for the confusion.

So then the actual request would be to get the resume_at field through the API, but that would be somewhere else, I'll try to find the correct forum for that request.