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

Enable progress reporting for ios. #11

Merged
merged 1 commit into from
Aug 7, 2023

Conversation

nvahalik
Copy link
Contributor

Modifies the SimpleSessionExporter to expose a progress Float value. This Float value defaults to 0 if there is no exportSession. A new exportSession private var was added to store the current exportSession. The AVAssetExportSession completion handler updates the exportSession reference to nil once it has completed.

To actually send the updates, we add a schedule job every 250 milliseconds which calls our progressHandler. The exporter completion handler cancels it when it is done.

This PR aims to address issue #10

Modifies the SimpleSessionExporter to expose a progress Float value. This Float value defaults to 0 if there is no exportSession. A new exportSession private var was added to store the current exportSession.
The AVAssetExportSession completion handler updates the exportSession reference to nil once it has completed.

To actually send the updates, we add a schedule job every 250 milliseconds which calls our progressHandler. The exporter completion handler cancels it when it is done.
@dragermrb dragermrb merged commit 980df2b into dragermrb:main Aug 7, 2023
@dragermrb
Copy link
Owner

Thanks @nvahalik

Merged and version v5.0.3 released!

@nvahalik nvahalik deleted the ios-progress-update-handler branch September 25, 2023 13:58
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