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

Feature/Add chatType option for fetching "LiveChat" #94

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

advancedbear
Copy link

Issue #80 でも触れられている「TopChat」ではなく「LiveChat」を取得する方法について、コメントにて指摘されている通りYouTubeページから取得できるcontinuationは2種類あり、そのうち一方がTopChat用のcontinuation、もう一方がLiveChat用のcontinuationとなっているようです。

今回、LiveChatクラスのコンストラクタ引数として新たに chatType を作成し、trueが指定された場合には2つ目のcontinuation(=LiveChat用のcontinuation)が選択されるように変更しました。

手元の環境で build し、それぞれの continuation で期待する動作をすることは確認しております。
(取得件数としても LiveChat > TopChat となるようです)

@advancedbear
Copy link
Author

#96 の問題を解消するため、新たに LiveChatMembershipMilestoneRenderer インターフェイスを定義し、適切にmessageを抽出できるように修正しました。

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.

1 participant