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站视频支持av/bv号 #287

Merged
merged 3 commits into from
Feb 19, 2025
Merged

嵌入B站视频支持av/bv号 #287

merged 3 commits into from
Feb 19, 2025

Conversation

Jinvic
Copy link
Contributor

@Jinvic Jinvic commented Feb 18, 2025

这个 PR 做了什么?

fix: #263

原来只支持使用如下格式的嵌入代码:

<iframe src="//player.bilibili.com/player.html?isOutside=true&aid=88602528&bvid=BV1U7411V78R&cid=151350705&p=1" scrolling="no" border="0" frameborder="no" framespacing="0" allowfullscreen="true"></iframe>

现在可以使用av/bv号嵌入视频了,直接填入av/bv号或者带av/bv号的视频链接都行。

测试镜像:jinvic/moments:test2

示例:

226a2bfcdbad10f97ae655798ff55066
ed69d146b7eb7ab05e15c4452db86eac

@Jinvic Jinvic marked this pull request as draft February 18, 2025 08:24
@Jinvic
Copy link
Contributor Author

Jinvic commented Feb 19, 2025

已更新pr

将处理后的url存入数据库,而不是存入原始数据,使用时再处理。

数据库迁移示例:

image

测试镜像:jinvic/moments:test2
注意: 将迁移数据库,请提前做好备份。

@Jinvic Jinvic marked this pull request as ready for review February 19, 2025 00:53
@xuewenG
Copy link
Collaborator

xuewenG commented Feb 19, 2025

辛苦了,今天晚上会 review 一下

@xuewenG xuewenG merged commit a149397 into kingwrcy:dev Feb 19, 2025
@xuewenG xuewenG added the bugfix Bug fix label Feb 19, 2025
@xuewenG xuewenG added this to the v0.2.9 milestone Feb 19, 2025
@xuewenG xuewenG added feature New feature and removed bugfix Bug fix labels Feb 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

无效的B站视频地址
2 participants