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(ios): migrate from deprecated methods #3444

Conversation

KrzysztofMoch
Copy link
Collaborator

@KrzysztofMoch KrzysztofMoch commented Dec 26, 2023

Summary

Part of #3425
Migrate AVAsset.tracks(withMediaType:) and AVAsset.mediaSelectionGroup(forMediaCharacteristic:) to newer equivalents

The diff on Github is pretty weird, so I recommend checking it in the editor

Why ?

To support visionOS and to improve performance by using async methods

Test plan

  • Tested in example app
  • CI passed

@KrzysztofMoch KrzysztofMoch force-pushed the feat/migrate-from-deprecated-methods branch from df8b06a to 6211b4d Compare December 26, 2023 11:39
@KrzysztofMoch KrzysztofMoch mentioned this pull request Dec 26, 2023
16 tasks
@KrzysztofMoch KrzysztofMoch force-pushed the feat/migrate-from-deprecated-methods branch 6 times, most recently from 723cda7 to d1f29ee Compare December 26, 2023 14:06
@freeboub
Copy link
Collaborator

freeboub commented Jan 4, 2024

@KrzysztofMoch I approve the Pr, but I don't have time to test it for now.
Up to you to merge it !

@KrzysztofMoch KrzysztofMoch force-pushed the feat/migrate-from-deprecated-methods branch 2 times, most recently from 6400615 to d1f29ee Compare January 6, 2024 17:48
@KrzysztofMoch KrzysztofMoch force-pushed the feat/migrate-from-deprecated-methods branch from d1f29ee to 9356c7f Compare January 6, 2024 18:40
@KrzysztofMoch KrzysztofMoch merged commit 5aaa53d into TheWidlarzGroup:master Jan 6, 2024
6 checks passed
@KrzysztofMoch KrzysztofMoch deleted the feat/migrate-from-deprecated-methods branch January 6, 2024 19:07
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.

None yet

2 participants