-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
plugins.chzzk: add clips support #6389
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
plugins.chzzk: add clips support #6389
Conversation
cb4c64a
to
9aaec63
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks.
Please remove the unrelated commit from the PR branch.
9aaec63
to
8344edc
Compare
8344edc
to
8da34ae
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes should be fine now. The order of returned tuple items in get_videos()
and get_clips()
is now different from get_live_detail()
, but that's not a big deal. If you want to change and revert it to the previous order for the sake of consistency, you can do that, otherwise I'm going to merge the PR.
Thanks.
I think I'll leave the order as it is for now. |
Esteemed expert, might I inquire as to how this API was discovered? https://api.chzzk.naver.com/service/v1/play-info/clip/{clip_id} Upon opening https://chzzk.naver.com/clips/LeqAmSuEQy in Google Chrome, I did not observe this API within the devtools. The only APIs of this nature that I encountered were: https://api.chzzk.naver.com/service/v1/clips/vDHdB5Jfl1/detail?optionalProperties=COMMENT&optionalProperties=PRIVATE_USER_BLOCK&optionalProperties=PENALTY&optionalProperties=MAKER_CHANNEL&optionalProperties=OWNER_CHANNEL. I eagerly await your response. |
Is this link obtained by visiting https://chzzk.naver.com/clips/1niF8oNcXt and then clicking the share button?Thank you very much. |
Closes: #6333
before merging, would be cool resolve PR #6388.