-
Notifications
You must be signed in to change notification settings - Fork 1
fix: update sdk name #22
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
Conversation
CHANGELOG.md
Outdated
| - Added screenshare functionality on both iOS and Android. Screensharing can be enabled in a room with `toggleScreenSharing(true)` and disabled with `toggleScreenSharing(false)` | ||
| - Added new callbacks **onRoomIsReconnecting** and **onRoomDidReconnect** (Android & iOS) to detect signalling interruptions and successful reconnections. | ||
| - Made **data track optional** on both iOS and Android. Data track now behaves like audio and video tracks: | ||
| - Made **data track optional** on both iOS and Android. Data track now behaves like audio and video tracks. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it'd be nice to explain what options/methods can be used to control data tracks now (enableData when connecting, setLocalDataTrackEnabled in the room)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was not addressed
CHANGELOG.md
Outdated
| - Added screenshare functionality on both iOS and Android. Screensharing can be enabled in a room with `toggleScreenSharing(true)` and disabled with `toggleScreenSharing(false)` | ||
| - Added new callbacks **onRoomIsReconnecting** and **onRoomDidReconnect** (Android & iOS) to detect signalling interruptions and successful reconnections. | ||
| - Made **data track optional** on both iOS and Android. Data track now behaves like audio and video tracks: | ||
| - Made **data track optional** on both iOS and Android. Data track now behaves like audio and video tracks. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was not addressed
|
Please update the PR description to reflect the changes that were made. |
|
One more thing: let's add a changelog entry to reflect the change in the npm package name:
|
Co-authored-by: Girts Rudziss <girtsrudziss@gmail.com>
Changes
Contributing to Twilio