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

Release 4.5.0, merge feature/media-optimizations #169

Merged
merged 6 commits into from
Aug 27, 2021

Conversation

ceaglest
Copy link
Contributor

@ceaglest ceaglest commented Aug 27, 2021

Update video-app-ios to take advantage of new APIs in 4.5.0 clientTrackSwitchOffMode and contentPreferencesMode and migrate off the deprecated APIs maxTracks and renderDimensions.

Testing

  • Building on CI

Contributing to Twilio

All third-party contributors acknowledge that any contributions they provide will be made under the same open-source license that the open-source project is provided under.

  • I acknowledge that all my contributions will be made under the project's license.

ceaglest and others added 4 commits May 28, 2021 13:25
* Client track switch off is enabled by default
* The app does not implement any specific manual controls, this is effectively the legacy behavior without maxTracks
* Remove renderer when cell is not visible
* Remove maxTracks from the app.
* Use 4.5.0-rc1.
* Update pods
* Remove renderer when video track changes
* Fix podfile

Co-authored-by: Tim Rozum <trozum@twilio.com>
* Use 4.5.0-rc4. [skip ci]

* Add support for video content preferences.

* Ability to select VCP in settings
* ConnectOptionsFactory applies the settings
* SettingsStore provides storage / default value

* Remove unused VideoDimensionsName class.

* Commit missing factory.

* Use mock generator to fix mock

* Fix pods

Co-authored-by: Tim Rozum <trozum@twilio.com>
Merge master into feature/media-optimization
@ceaglest ceaglest requested a review from timrozum August 27, 2021 00:27
@@ -1,3 +1,5 @@
source 'https://cdn.cocoapods.org/'
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not required but nice to be explicit, IMHO.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah this way we don't have to change this when switching between public and private pod. I thoughts that how it was but it must have gotten removed somehow.

Podfile Outdated Show resolved Hide resolved
Copy link
Contributor

@timrozum timrozum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I left one comment but I'm not worried about it.

@ceaglest ceaglest merged commit b9239f0 into master Aug 27, 2021
@timrozum timrozum deleted the feature/media-optimization branch May 20, 2022 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants