mikedilger / gossip

Gossip is a nostr client
Other
713 stars 81 forks source link

Fail to show video URL #909

Closed dtonon closed 1 day ago

dtonon commented 1 day ago

https://njump.me/nevent1qqsy7xjyu9wsp979askq7fana7yxg8v0cv30dehf9ew7v82ezg5tx4spramhxue69uhkymmnw3ezumrfva58gmnfdenhxur0wfjjucm0d5hsyuuvcf

The video URL is missing on Gossip. My build is compiled without the ffmpeg support. I cannot test the support since currently I'm getting a compilation error.

image
mikedilger commented 1 day ago

I just updated rust-ffmpeg and egui-video to support ffmpeg 7.1 (which also was giving me the compilation error). So you can now build with ffmpeg support again.

But the issue you bring up is surely something I broke. I updated media loading quite a lot to verify SHA signatures and to render blurhashes while waiting. In that process I believe the [ Video ] link was removed in some cases (e.g. when the video itself was available (it has a link on the right-hand side)). Maybe it got removed in a case that still matters and needs it.

mikedilger commented 1 day ago

Fixed on unstable