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(telegram): correct video behavior #332

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

std-microblock
Copy link
Contributor

No description provided.

@std-microblock
Copy link
Contributor Author

This fixture is dirty as it depends on system ffprobe/ffmpeg directly. idk if there's a way to download it instead. @Anillc any idea?

@std-microblock std-microblock marked this pull request as draft December 27, 2024 18:42
@Anillc
Copy link
Contributor

Anillc commented Dec 28, 2024

为什么要自己生成 thumbnail, 它不是 optional 吗

另外 ffmpeg 可以在内存里完成操作吧,应该不用写到文件去

感觉可以把 getThumbnail 的操作放到 utils.ts 之类的地方,都放 message.ts 太难受了

ffmpeg 插件可以下 ffmpeg 但是我印象中有平台没有 ffprobe。最近我会重新写一下 downloads 和 ffmpeg 插件,之前写得好糟糕啊)

@std-microblock
Copy link
Contributor Author

不生成的话会 1. 视频从外面看起来是黑的 2. 必须要下完才能放

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.

2 participants