Skip to content
This repository was archived by the owner on Jun 23, 2023. It is now read-only.

add playlist and auto-start#4

Open
develop-branch-2020 wants to merge 1 commit intoPRNDcompany:mainfrom
develop-branch-2020:feat_playlist
Open

add playlist and auto-start#4
develop-branch-2020 wants to merge 1 commit intoPRNDcompany:mainfrom
develop-branch-2020:feat_playlist

Conversation

@develop-branch-2020
Copy link

add playlist and auto-start

wasRestored: Boolean
) {
if (!wasRestored) {
if (autoStart) {
Copy link
Contributor

@ted-prnd ted-prnd Dec 4, 2020

Choose a reason for hiding this comment

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

play(), playList()의 코드가 하는일이 똑같고 loadVideo()/cueVideo() or loadPlaylist()/cuePlaylist()만 다릅니다.
이를 처리하는 action을 함수 parameter로 받아서 처리하는 방식으로 개선해주세요

play(), playList() functions is almost same
That functions different only loadVideo()/cueVideo() or loadPlaylist()/cuePlaylist()
Change function parameter for action

Choose a reason for hiding this comment

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

안녕하세요 이 라이브러리를 말씀하신대로 수정하려고 하는데 이 파일이 read-only로 되어있는데 말씀하신대로 수정하려면 코드를 복붙해서 제가 새로운 라이브러리를 만들면 되나요? 초보적인 질문 죄송합니다.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants