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

feat: [breaking] add connection and socket timeouts on voice application #548

Merged
merged 14 commits into from
Mar 13, 2024

Conversation

Tr00d
Copy link
Contributor

@Tr00d Tr00d commented Oct 27, 2023

Description

Add missing connection and socket timeouts on Voice application.
This is a breaking change, given we need to get rid of the Capability inheritance, in order to add more properties only for Voice.

Copy link

sonarcloud bot commented Dec 18, 2023

Quality Gate Passed Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

6 New issues
0 Security Hotspots
92.3% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

# Conflicts:
#	Vonage.Test/Data/ApplicationTests/CreateApplicationAsyncWithVoiceTimeouts-request.json
#	Vonage.Test/Data/ApplicationTests/CreateApplicationAsyncWithVoiceTimeouts-response.json
Copy link

sonarcloud bot commented Jan 30, 2024

Quality Gate Passed Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

6 New issues
0 Security Hotspots
92.3% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@Tr00d Tr00d added the v7.0.0 label Jan 31, 2024
# Conflicts:
#	Vonage.Test/Vonage.Test.csproj
Copy link

sonarcloud bot commented Mar 8, 2024

@Tr00d Tr00d marked this pull request as ready for review March 13, 2024 09:39
@Tr00d Tr00d merged commit 8948ead into main Mar 13, 2024
5 checks passed
@Tr00d Tr00d deleted the breaking-application branch March 13, 2024 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant