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

Utils+Everywhere: Add a start offset tolerance for the external player #2326

Merged
merged 1 commit into from
Jun 19, 2022

Conversation

Svallinn
Copy link
Member

@Svallinn Svallinn commented Jun 16, 2022

Pull Request Type

  • Feature Implementation

Description
RC version of #2325

If a current video is practically over or the value in
the history's watch progress is almost the same as the length
of the video, it's safe to deduce that if the external player
button is clicked, the user is most likely just rewatching
the video, making no sense to pass a start offset to it
@PrestonN PrestonN enabled auto-merge (squash) June 16, 2022 16:13
PikachuEXE
PikachuEXE previously approved these changes Jun 17, 2022
absidue
absidue previously approved these changes Jun 17, 2022
Copy link
Member

@absidue absidue left a comment

Choose a reason for hiding this comment

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

Tested with VLC.

@absidue absidue added the PR: waiting for review For PRs that are complete, tested, and ready for review label Jun 17, 2022
auto-merge was automatically disabled June 19, 2022 18:41

Pull request was closed

@PrestonN PrestonN enabled auto-merge (squash) June 19, 2022 18:41
@github-actions github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Jun 19, 2022
@efb4f5ff-1298-471a-8973-3d47447115dc efb4f5ff-1298-471a-8973-3d47447115dc added the PR: waiting for review For PRs that are complete, tested, and ready for review label Jun 19, 2022
@efb4f5ff-1298-471a-8973-3d47447115dc efb4f5ff-1298-471a-8973-3d47447115dc changed the base branch from v0.17.0-RC to development June 19, 2022 18:54
@efb4f5ff-1298-471a-8973-3d47447115dc efb4f5ff-1298-471a-8973-3d47447115dc dismissed stale reviews from absidue and PikachuEXE June 19, 2022 18:54

The base branch was changed.

@efb4f5ff-1298-471a-8973-3d47447115dc efb4f5ff-1298-471a-8973-3d47447115dc changed the base branch from development to v0.17.0-RC June 19, 2022 18:57
@PrestonN PrestonN disabled auto-merge June 19, 2022 19:09
@PrestonN PrestonN merged commit c9c6b38 into v0.17.0-RC Jun 19, 2022
@github-actions github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Jun 19, 2022
@Svallinn Svallinn deleted the feat/external-player-start-offset-tolerance-RC branch June 19, 2022 22:04
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.

5 participants