status-mobile icon indicating copy to clipboard operation
status-mobile copied to clipboard

The static images are not shown in for collectible on collectible list

Open VolodLytvynenko opened this issue 1 year ago • 6 comments

Steps:

  1. Add a watch-only account using the address 0x8d2413447ff297d30bdc475f6d5cb00254685aae.
  2. Scroll through the list of collectibles.

Actual result:

An uploading spinner is always displayed for animated/audio/video collectibles, and static images are not shown.

https://github.com/status-im/status-mobile/assets/52490791/b8a1871d-395b-4340-8bac-9c597bb2e57e

Expected result:

The static snapshot should be displayed in the collectible list for animated/audio/video collectibles.

reference https://discord.com/channels/624634427930312714/1200162022189441104/1200390987038609418

ENV:

Nightly [25 Jan 2024]

VolodLytvynenko avatar Jan 25 '24 19:01 VolodLytvynenko

BLOCKED: At the moment, the data that are coming from the backend don't include a preview image for the videos, svgs, etc.

mmilad75 avatar Jan 26 '24 13:01 mmilad75

@mmilad75 do you mind if I take this one over? Already started on fixing the collectibles performance by generating preview images. This seems to be in the same realm of changes.

clauxx avatar Feb 13 '24 13:02 clauxx

@mmilad75 do you mind if I take this one over? Already started on fixing the collectibles performance by generating preview images. This seems to be in the same realm of changes.

Sure, you can take it over. @clauxx

mmilad75 avatar Feb 13 '24 13:02 mmilad75

@clauxx - can you update on why you unassigned this? What findings did you get in the end?

J-Son89 avatar May 13 '24 10:05 J-Son89

@J-Son89 i unassigned it due to a change in priorities. I was working on generating thumbnails with the media server (which seemed related to this task, hence i assigned this to myself), but we started working with crews and prepping for the release so I stopped working on it and unassigned myself. This specific task i didn't start on.

clauxx avatar May 13 '24 10:05 clauxx

thanks @clauxx

J-Son89 avatar May 13 '24 10:05 J-Son89

@VolodLytvynenko - I think we updated the collectible component since this. Is this issue still relevant?

J-Son89 avatar May 23 '24 22:05 J-Son89

@VolodLytvynenko - I think we updated the collectible component since this. Is this issue still relevant?

hi @J-Son89 Thank you for the update. The issue can be closed

VolodLytvynenko avatar May 27 '24 12:05 VolodLytvynenko