Skip to content

fix: Update region observer with new rules for small regions #8275

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

Merged
merged 2 commits into from
Mar 14, 2025

Conversation

avelad
Copy link
Member

@avelad avelad commented Mar 14, 2025

This is necessary because some EMSG and ID3 events might not fire because the endTime is equal to or very close to the startTime.

Related to #8012

This is necessary because some EMSG and ID3 events might not fire because the endTime is equal to or very close to the startTime.
Related to shaka-project#8012
@avelad avelad added type: bug Something isn't working correctly priority: P1 Big impact or workaround impractical; resolve before feature release labels Mar 14, 2025
@avelad avelad added this to the v4.15 milestone Mar 14, 2025
@shaka-bot
Copy link
Collaborator

shaka-bot commented Mar 14, 2025

Incremental code coverage: 87.50%

Copy link
Member

@tykus160 tykus160 left a comment

Choose a reason for hiding this comment

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

Perfecto!

@avelad
Copy link
Member Author

avelad commented Mar 14, 2025

Perfecto!

Dziękuję!

@avelad avelad merged commit 2f18b47 into shaka-project:main Mar 14, 2025
30 of 32 checks passed
avelad added a commit that referenced this pull request Mar 14, 2025
This is necessary because some EMSG and ID3 events might not fire
because the endTime is equal to or very close to the startTime.

Related to #8012
avelad added a commit that referenced this pull request Mar 14, 2025
This is necessary because some EMSG and ID3 events might not fire
because the endTime is equal to or very close to the startTime.

Related to #8012
@avelad avelad deleted the missing-emsg-id3-events branch March 17, 2025 10:15
@shaka-bot shaka-bot added the status: archived Archived and locked; will not be updated label May 13, 2025
@shaka-project shaka-project locked as resolved and limited conversation to collaborators May 13, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
priority: P1 Big impact or workaround impractical; resolve before feature release status: archived Archived and locked; will not be updated type: bug Something isn't working correctly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants