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

Releases/v4.2.0 #255

Merged
merged 9 commits into from
Nov 23, 2024
Merged

Releases/v4.2.0 #255

merged 9 commits into from
Nov 23, 2024

Conversation

andrewjl-mux
Copy link
Contributor

@andrewjl-mux andrewjl-mux commented Nov 22, 2024

Fixes

Co-authored-by: AJ Lauer Barinov [email protected]
Co-authored-by: Emily Dixon [email protected]

@andrewjl-mux andrewjl-mux requested a review from a team as a code owner November 22, 2024 21:07
Copy link
Collaborator

@daytime-em daytime-em left a comment

Choose a reason for hiding this comment

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

I found a last-minute issue, just a sec

@daytime-em
Copy link
Collaborator

daytime-em commented Nov 22, 2024

PR for the issue: #256

edit:also fixes the build

Copy link
Collaborator

@daytime-em daytime-em left a comment

Choose a reason for hiding this comment

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

ok fix added. I don't think we need to note it because the app-termination thing is new to this version

@daytime-em
Copy link
Collaborator

(Approved but we still need to update the XCFramework)

* Revert "fix: clear the app-termination observer when we dealloc (#256)"

This reverts commit 719cc7b.

* Revert "fix: tear down view on hard application termination (#249)"

This reverts commit 71befd8.

* still want to get rid of the NSLogs
@andrewjl-mux andrewjl-mux merged commit 87aada2 into master Nov 23, 2024
3 checks passed
@andrewjl-mux andrewjl-mux deleted the releases/v4.2.0 branch November 23, 2024 01:47
Copy link

A draft release has been created for this version. Find it here! https://github.com/muxinc/mux-stats-sdk-avplayer/releases/tag/untagged-87c7a56698cd9ea84bf7

andrewjl-mux added a commit that referenced this pull request Nov 23, 2024
* fix: send `ended` when an AVPlayerItem finishes playing to completion (#254)

* fix: remove NSLog and comment lines
---------

Co-authored-by: Emily Dixon <[email protected]>
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