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

Previews of videos are not visible in messageList on Android, but on iOS yes #3369

Closed
JeromeCHA opened this issue Apr 22, 2022 · 8 comments · Fixed by #4058 or #4158
Closed

Previews of videos are not visible in messageList on Android, but on iOS yes #3369

JeromeCHA opened this issue Apr 22, 2022 · 8 comments · Fixed by #4058 or #4158
Assignees

Comments

@JeromeCHA
Copy link

Describe the bug
When you send a video via Stream on a channel, you don't have any preview on Android, but on iOS yes. (see screenshots below)

SDK version

  • 5.0.2

To Reproduce
Steps to reproduce the behavior:

  1. Click on the attachment button on the input view
  2. Choose a video
  3. Send it
  4. Check on both Android and iOS and see the difference

Expected behavior
It would be glad if we could also have a preview of the video on the message list. It would avoid both apps to have different designs and also different behaviors. It is also more convenient to have the preview rather than the filename so we can figure out what the video is.

Device:

  • Vendor and model: Pixel 3
  • Android version: Android 12

Screenshots

iOS Android
file Screenshot_20220422_103554
@JeromeCHA JeromeCHA added the bug Something isn't working label Apr 22, 2022
@MarinTolic
Copy link
Contributor

Hey Jerome,

Thank you for reporting this discrepancy.

Video thumbnail previews are indeed on the roadmap for the Android SDKs, however we are currently waiting for certain backend optimizations before we can proceed with the work. Once we implement this we will notify you. :D

In the meantime I will assign this issue to myself and change it from a bug to a feature request.

Kind regards,
Marin.

@MarinTolic MarinTolic added feature request enhancement New feature or request and removed bug Something isn't working labels Apr 22, 2022
@MarinTolic MarinTolic self-assigned this Apr 22, 2022
@kylecfoley
Copy link

Hey @MarinTolic,

Is there someone we could speak with at Stream regarding moving this up on the priority list? It will have a significant impact on our chat experience (users will be paying to unlock video content within the chat).

Thanks for your help,
Kyle

@filbabic
Copy link
Contributor

Hey @kylecfoley !

We'll see what we can do in terms of the roadmap - in the case, as Marin mentioned, it's a change for the BE team as well and their roadmap is pretty packed. Hopefully we can work something out to have this support ASAP.

As an alternative, we'll look into providing a guide on how to build this custom message item type so that you're not blocked by the request.

As far as speaking to someone from the team about shifting the priority here, we'll look into that as well. We'll let you know as soon as we know more info!

Thanks!

@kylecfoley
Copy link

Hey @filbabic ! Regarding your note on providing a guide to build custom message type, that would be great :) Thanks

@JeromeCHA
Copy link
Author

Hey @filbabic, regarding the guide to build a custom message type that would help us, it would be awesome to have it with your UI Component, not Compose UI Component 🙇
Thank you very much 🙏

@filbabic
Copy link
Contributor

filbabic commented May 9, 2022

Hey @JeromeCHA @kylecfoley !

We'll see if we're able to provide something soon, as we are working through the rest of the items you suggested as well as some other higher priority items.

I'll keep you posted as soon as I know more! :]

@JayMehta97
Copy link

Hey @filbabic @MarinTolic !

Any updates on this feature or on the guide providing how to build this custom message item type?

Thanks.

@filbabic
Copy link
Contributor

filbabic commented Jul 7, 2022

Hey @JeromeCHA!

Thank you for following up - we were actually waiting for the BE impl which should've been ready but we've also had some movement in the company and higher priority items so we couldn't have commited to this directly 😞

There's lots going on and we're still to plan Q3s properly, so once I create the neccessary infrastructure, I'll add this to a sprint so we can focus on that.

Thank you so much for your patience and being understanding, it's been a rough month or so and we really do appreciate it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment