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

Restriction key must not include timestamp parameter #80

Open
mauriciofierrom opened this issue Dec 12, 2024 · 0 comments
Open

Restriction key must not include timestamp parameter #80

mauriciofierrom opened this issue Dec 12, 2024 · 0 comments
Labels
bug Something isn't working YouTubePlayer Related to the specific YouTube video player

Comments

@mauriciofierrom
Copy link
Owner

When playing a section, the t=n query param is sometimes appended to the key used to store the timestamp for manual play check for the YouTube player in local storage. This makes for duplicate entries and checks failure.

We should either make sure only the video id param is included v=<video-id> or change this part of the key altogether (perhaps to use only the video id and not the whole URL).

@mauriciofierrom mauriciofierrom added bug Something isn't working YouTubePlayer Related to the specific YouTube video player labels Dec 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working YouTubePlayer Related to the specific YouTube video player
Projects
None yet
Development

No branches or pull requests

1 participant