Manage endpoints updated to support Tokens - Githubissues

deepgram / deepgram-python-sdk

Official Python SDK for Deepgram's automated speech recognition APIs.

https://developers.deepgram.com

MIT License

235 stars 63 forks source link

Manage endpoints updated to support Tokens #170

Closed jpvajda closed 9 months ago

jpvajda commented 1 year ago

Manage endpoints that will likely change:

request - https://developers.deepgram.com/reference/get-request
requests - https://developers.deepgram.com/reference/get-all-requests
usage - https://developers.deepgram.com/reference/summarize-usage
fields - https://developers.deepgram.com/reference/get-fields"

UniversalDevGenius commented 9 months ago

@jpvajda I would like to work on this issue, can you provide more context regarding it.

UniversalDevGenius commented 9 months ago

@UniversalDevGenius You missed it by about a day... it's already implemented here: https://github.com/dvonthenen/deepgram-python-sdk/tree/feature-sits-impl

Just need to rename a couple of functions after discussing what they should be called.

@dvonthenen Can you share me some issues that I can contribute just tag them here or assign them to me, I really appreciate if you give me an chance to work on interesting issue to start mine first every open source contribution.

dvonthenen commented 9 months ago

hey @UniversalDevGenius drop me a line in Discord (https://dpgr.am/discord) if you are interested and we can discuss. The APIs for 3.1 are mostly finalized and the deadline is fast approaching.

UniversalDevGenius commented 9 months ago

hey @UniversalDevGenius drop me a line in Discord (https://dpgr.am/discord) if you are interested and we can discuss. The APIs for 3.1 are mostly finalized and the deadline is fast approaching.

Responded to you on Discord as well. Thanks for responding. Looking forward to discuss more on it.

dvonthenen commented 9 months ago

I think this is mostly implemented and was available in v3.0.0... might need to investigate some: https://github.com/deepgram/deepgram-python-sdk/blob/main/deepgram/clients/manage/v1/client.py#L621-L747

dvonthenen commented 9 months ago

Looks like new fields on the response objects. That's about the extent of the work.

dvonthenen commented 9 months ago

@UniversalDevGenius

for the support endpoint tokens (https://github.com/deepgram/deepgram-python-sdk/issues/170), I will double check... but you should just need to enable the debug flag in the example (https://github.com/deepgram/deepgram-python-sdk/blob/main/examples/manage/usage/main.py) and then grab a copy of the response JSON (in the debug statements it will be the output starting with json:) and then run the json through https://jsonformatter.org/json-to-python to copy over the new fields in the response objects

let me know if you have any questions or are unable to work on this. thanks for your help!

dvonthenen commented 9 months ago

PR Merged https://github.com/deepgram/deepgram-python-sdk/pull/259. Will be available on the next release (today or tomorrow-ish).