flathub / us.zoom.Zoom

https://flathub.org/apps/details/us.zoom.Zoom
36 stars 48 forks source link

style: update screenshots #325

Closed yajo closed 1 year ago

yajo commented 2 years ago

Fix https://github.com/flathub/us.zoom.Zoom/issues/295.

flathubbot commented 2 years ago

Started test build 89189

flathubbot commented 2 years ago

Build 89189 successful To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/86997/us.zoom.Zoom.flatpakref
nedrichards commented 2 years ago

If we're going to have non 'hosted on the upstream website' screenshots for an app like this then I think we need them in this repo as well since if they go away they will do very annoying things to builds, websites and app stores.

yajo commented 2 years ago

The image links are borrowed from this comment: https://github.com/flathub/us.zoom.Zoom/issues/295#issue-1051344536

I've never seen github to delete images attached to a comment. Do you really think that is necessary?

camelCaseNick commented 2 years ago

@yajo They are hosted upstream: https://explore.zoom.us/media/img-modernizemeeting_69f1e3a.png and https://explore.zoom.us/media/zfh-presentation.png.

flathubbot commented 2 years ago

Started test build 102808

flathubbot commented 2 years ago

Build 102808 successful To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/100516/us.zoom.Zoom.flatpakref
violetmage commented 1 year ago

bump

Can someone merge this?

flathubbot commented 1 year ago

Started test build 25718

muelli commented 1 year ago

yeah. (External) Screenshots tend to be a liability because it's a pain when they go away, but we currently have only one screenshot and I think the addition is warranted. Thanks!

flathubbot commented 1 year ago

Build 25718 successful To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/8377/us.zoom.Zoom.flatpakref