-
Notifications
You must be signed in to change notification settings - Fork 824
Feat/video message #801
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
base: main
Are you sure you want to change the base?
Feat/video message #801
Conversation
|
@demchenkoalex Putin this on draft to get your input, as integrating in my app, especially with a thumbnail raised quite a lot of questions I wanna discuss with you.
final fullSizeimageBytes =
await VideoThumbnail.thumbnailData(
video: videoUrl,
imageFormat: ImageFormat.WEBP,
quality: 1,
);
highResCover= base64.encode(fullSizeimageBytes);
// Then pass it to VideoMessage
|
b8a649f to
a983d1d
Compare
827803b to
4063c84
Compare
|
This PR is ready too. Like i said the
|
… HiRes images on his side)
cb9cea2 to
308213d
Compare
|
Is this ready to use? I might just take a couple of your patches onto a fork of mine to use, if that's okay? |
|
@Henry-Hiles i'm using it without any issue (but my app does not have many users). Code-wise it's ok, I was mostly discussing the default behavior/interface |
|
PS: (i may need some adjustement if the main branch moved a lot, did not pull in a while) |
Implementation notes
Maybe remaining points
ChatMessage(at least on the bubble). But if we want to prevent that the user would have to handle video playback.Recording.at.2025-06-03.00.17.18.mp4