Closed SecondeJK closed 1 year ago
Given I'll be traveling from tomorrow, I'm going to preapproved the PR to make sure you don't get blocked in the following days
Merging #417 (bbf57c2) into main (8e9f239) will increase coverage by
0.51%
. The diff coverage is87.78%
.
@@ Coverage Diff @@
## main #417 +/- ##
============================================
+ Coverage 77.97% 78.48% +0.51%
- Complexity 2116 2183 +67
============================================
Files 200 209 +9
Lines 5634 5894 +260
============================================
+ Hits 4393 4626 +233
- Misses 1241 1268 +27
Impacted Files | Coverage Δ | |
---|---|---|
src/Client/Exception/Request.php | 0.00% <ø> (ø) |
|
src/Client/Exception/ThrottleException.php | 100.00% <ø> (ø) |
|
src/Client/Exception/Validation.php | 100.00% <ø> (ø) |
|
src/Meetings/ClientFactory.php | 0.00% <0.00%> (ø) |
|
src/Entity/IterableAPICollection.php | 71.73% <33.33%> (-1.37%) |
:arrow_down: |
src/Meetings/DialInNumber.php | 33.33% <33.33%> (ø) |
|
src/Meetings/Application.php | 66.66% <66.66%> (ø) |
|
src/Meetings/ApplicationTheme.php | 66.66% <66.66%> (ø) |
|
src/Meetings/Recording.php | 66.66% <66.66%> (ø) |
|
src/Meetings/Room.php | 66.66% <66.66%> (ø) |
|
... and 4 more |
... and 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
This PR adds support for the Meetings API
Description
New client with all relevant methods documented
Motivation and Context
Evolution of Vonage APIs
How Has This Been Tested?
Full test suite included
Types of changes
Checklist: