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

fix: 7.28.0+ compatibility #1277

Merged
merged 6 commits into from
Sep 29, 2023
Merged

fix: 7.28.0+ compatibility #1277

merged 6 commits into from
Sep 29, 2023

Conversation

cxw620
Copy link
Collaborator

@cxw620 cxw620 commented Sep 28, 2023

fix:

@yujincheng08
Copy link
Owner

MusicNotification 感觉可以不要了,b 站已经是原生的通知了

@cxw620
Copy link
Collaborator Author

cxw620 commented Sep 28, 2023

MusicNotification 感觉可以不要了,b 站已经是原生的通知了

直接 remove? 还是丢在那不管了

@yujincheng08
Copy link
Owner

感觉可以直接扔掉

@cxw620
Copy link
Collaborator Author

cxw620 commented Sep 28, 2023

就 B 这天天改东西, 也不知道啥时候稳定下来能发版 1.6.13...
行, 那我扔掉了

@cxw620
Copy link
Collaborator Author

cxw620 commented Sep 28, 2023

7280200 测试没问题, 我看看 7.29
寄, 这是什么玩意, 不过看起来是个例? 修好了

m

@cxw620 cxw620 marked this pull request as draft September 28, 2023 07:04
@cxw620 cxw620 marked this pull request as ready for review September 28, 2023 07:24
@cxw620 cxw620 marked this pull request as draft September 28, 2023 07:31
@cxw620 cxw620 marked this pull request as ready for review September 28, 2023 07:44
@yujincheng08 yujincheng08 enabled auto-merge (squash) September 28, 2023 19:08
@cxw620
Copy link
Collaborator Author

cxw620 commented Sep 29, 2023

应该修完了, 可以 review 了

@yujincheng08 yujincheng08 merged commit 978bb1b into yujincheng08:master Sep 29, 2023
2 checks passed
@PianCat
Copy link

PianCat commented Oct 3, 2023

还是不大行,从隔壁频道获取的7.48和7.49都没有原生音乐通知栏。
全局又老又旧

@mio-19
Copy link

mio-19 commented Oct 14, 2023

MusicNotification 感觉可以不要了,b 站已经是原生的通知了

Play版还不是原生的通知,请求重新加入

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants