Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove deepgram.listen, deepgram.manage|onprem, etc Dot Notation #378

Open
davidvonthenen opened this issue Apr 24, 2024 · 0 comments
Open
Labels
low priority Do these last...

Comments

@davidvonthenen
Copy link
Contributor

Proposed changes

We should just require users to directly instantiate the classes and no longer have the Dot Notation mechanism. This is more or less a Javascript construct that we shouldn't carry forward in the next major version.

We had to ditch this in the .NET SDK because of the way .NET packaging works. It's actually what contributed to the "global" timeout issue here:
deepgram/deepgram-dotnet-sdk#207 (comment)

Context

Na

Possible Implementation

NA

Other information

NA

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
low priority Do these last...
Projects
None yet
Development

No branches or pull requests

2 participants
@davidvonthenen and others