PWA improvements, native version & ability to use specific version

icidasset / diffuse

A music player that connects to your cloud/distributed storage.

https://diffuse.sh

Other

807 stars 67 forks source link

PWA improvements, native version & ability to use specific version #273

Closed icidasset closed 3 years ago

icidasset commented 3 years ago

The app should be installable as a PWA already, and offline available. I want to move away from the native version, because (a) it's ineffecient, 100+ MB download, and (b) I can't build the linux and windows versions anymore.

To do:

[x] Make sure the PWA is working properly
[x] Explore https://web.dev/web-bundles/ & https://github.com/WICG/webpackage
[x] Give https://github.com/tauri-apps/tauri-action another shot (see Github action)
[x] ~~Try out https://wails.app/~~
[x] ~~Figure out how to make the app "upgradeable", so people can stick with a specific version~~ (Download a specific release from Github and host it yourself)

Netherquark commented 3 years ago

I am happy to report the PWA is working perfectly offline The only thing I noticed is that a different (I think default) wallpaper loads instead of the selected one when Diffuse is opened offline Also, after the following sequence, Diffuse takes a lot of time to open:

Open Diffuse offline
Close Diffuse
Restore Internet
Open Diffuse with internet working

Everything I tested rn worked perfectly though And kudos for the lightning-fast load times offline

Edit: Okay everything is working perfectly on Windows for me, but the no tracks error is still there on Chrome on Android

icidasset commented 3 years ago

@Netherquark Nice, thanks for testing that! 🙏

I do have to rewrite the service worker so it uses the cached app files and fetches the new ones in the background (when online), instead of always fetching the new files. ~~Will do this soon.~~ Done.

icidasset commented 3 years ago

PWA is working fine
Tried making a native build with https://tauri.studio But it doesn't work because Safari doesn't allow running service workers while using a custom protocol (ie. tauri://) Everything is in place though, src-tauri and the Github action. Maybe this'll get fixed eventually.

Netherquark commented 3 years ago

This is still happening on android so I don't think the no tracks issue is rectified yet.

icidasset commented 3 years ago

Thanks @Netherquark I'll delve into more specific issues later.