Skip to content

Amend position attribute in startResumePlayback method #107

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ed1001
Copy link

@ed1001 ed1001 commented Feb 25, 2024

PlayerEndpoints: Amend the positionMs param/body attribute in startResumePlayback to the correct: position_ms

Problem

The param and body attribute positionMs in the startResumePlayback method in PlayerEndoints.ts is camelCase whereas the documentation says it should be snake_case.

Solution

Rename the param and attribute to position_ms

Result

Playback will start/resume at the specified postion

@itsamaik
Copy link

Just ran into this as well. Would be great to see this merged.

@philheller
Copy link

philheller commented May 18, 2024

+1, would be great, if this could be merged👍🏽

@itsamaik
Copy link

@jordan-loeser @zhammer Hi guys, could you please review this PR and merge it? It would save an extra API call.

@danielvy
Copy link

It's a simple fix, please merge this PR.

@ed1001
Copy link
Author

ed1001 commented Jul 14, 2024

yes please can someone merge this, I don't have the access.

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.

4 participants