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

增加了对B站8K超高清视频的支持 #978

Merged
merged 16 commits into from
Dec 7, 2021

Conversation

LazzyFatty
Copy link
Contributor

@LazzyFatty LazzyFatty commented Dec 6, 2021

目前B站只在电视端和Safari浏览器支持8k视频,所以设置了了一个UA,那个地方如果后续B站进一步开放了8k的设备,可以去掉。

此外,无关本次修改的代码,我还有一个小问题,是我的操作哪里存在问题吗?我的每一次拉取请求都会带上从我第一次贡献代码开始的所有提交,尽管之前那些提交已经被合并

close #977

@LazzyFatty
Copy link
Contributor Author

我意识到我犯了一个错误,我在修改代码之前忘记先获取上游的修改了,请问我现在应该如何解决

@iawia002
Copy link
Owner

iawia002 commented Dec 6, 2021

我意识到我犯了一个错误,我在修改代码之前忘记先获取上游的修改了,请问我现在应该如何解决

你先 rebase 一下你的分支,再把 1269714 这个 commit cherry-pick 过去

@LazzyFatty
Copy link
Contributor Author

我意识到我犯了一个错误,我在修改代码之前忘记先获取上游的修改了,请问我现在应该如何解决

你先 rebase 一下你的分支,再把 1269714 这个 commit cherry-pick 过去

非常抱歉,给您添麻烦了,我现在已经解决完我的分支了,您可以开始检查本次提交了

Copy link
Owner

@iawia002 iawia002 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@iawia002 iawia002 merged commit 950db57 into iawia002:master Dec 7, 2021
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.

Site feature request - 8K video support for bilibili.com
2 participants