Closed manchuck closed 1 year ago
@manchuck whats still left in this PR to wrap up? Would love to see this get merged 🚢
@ziahamza We have put this release on hold as our internal team is working on changes to the API endpoints, which might not be compatible with the current meetings API specification. Once those changes have been released, we will work on updating the SDK.
Merging #788 (2c3ccb3) into 3.x (e50c7cb) will increase coverage by
1.51%
. The diff coverage is97.54%
.
@@ Coverage Diff @@
## 3.x #788 +/- ##
==========================================
+ Coverage 80.06% 81.57% +1.51%
==========================================
Files 99 100 +1
Lines 1289 1411 +122
Branches 257 280 +23
==========================================
+ Hits 1032 1151 +119
- Misses 256 259 +3
Partials 1 1
Impacted Files | Coverage Δ | |
---|---|---|
packages/meetings/lib/meetings.ts | 97.52% <97.52%> (ø) |
|
packages/server-client/lib/client.ts | 98.43% <100.00%> (+0.02%) |
:arrow_up: |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
Adding the meetings API
Description
This adds the meetings API calls to the SDK.
Motivation and Context
Testing Details
Example Output or Screenshots (if appropriate)
Types of changes
Checklist