Closed dvonthenen closed 1 month ago
The recent changes involve the removal of two projects (websocket
and simple
) from the solution, adjustments to exception handling within the AbstractRestClientTests
, updates to constructors in exception classes to include default values, and removal of the "Text to Speech - WebSocket" section from the README. Additionally, the CreateSpeakWebSocketClient
method in ClientFactory
has been commented out, deactivating its functionality.
File Path | Change Summary |
---|---|
Deepgram.Dev.sln |
Removed projects websocket and simple from the solution configuration. |
Deepgram.Tests/UnitTests/ClientTests/…ClientTests |
Updated Delete_Should_Throws_HttpRequestException_On_Response_Containing_Error to throw HttpRequestException instead of DeepgramException . |
Deepgram/ClientFactory.cs |
Commented out the CreateSpeakWebSocketClient method. |
Deepgram/Models/Exceptions/v1/DeepgramException.cs |
Added default values for ErrMsg , ErrCode , and RequestId in the DeepgramException constructor. |
Deepgram/Models/Exceptions/v1/DeepgramRESTException.cs |
Updated DeepgramRESTException constructor to include a default parameter for the error message. |
README.md |
Removed the "Text to Speech - WebSocket" section and its associated example link. |
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Temporarily Remove TTS WS Implementation. Will have a subsequent PR to be held for merge when the feature is ready.
Summary by CodeRabbit
New Features
Bug Fixes
HttpRequestException
instead ofDeepgramException
.Documentation
Chores
CreateSpeakWebSocketClient
method, effectively removing it from functionality.