cinny
cinny copied to clipboard
cinnyapp
Feature Request: PWA
I think this issue confuses PWA with just being responsive.
I think so PWA and responsive design are two different things....btw, what is the progress on any of this topic?
This might be interesting regarding this issue: https://developer.mozilla.org/en-US/docs/Web/API/Badging_API
PWA will allow Cinny to be installable on desktop or mobile and make it feel like a native app. Other things can be done like full offline support so it loads fully from the cache.
This might help. https://create-react-app.dev/docs/making-a-progressive-web-app/
I would like to see PWA support, as then I could use Cinny on my phone like a native app.
I assume we need a PWA service worker to get true push notification support on mobile besides caching, so this would be a killer feature.
I had been working on implementing background notifications using service worker, but it requires changes to how Cinny saves the token since localstorage, where it lives today, is not available in serviceworkers, so it would have to be migrated to indexeddb. If someone is interested to take it over: https://github.com/c0ffeecode/cinny/tree/background-notifications
It takes advantage of push gateways and ntfy, so it can listen for notifications using WebSockets efficiently.
As for the Tauri build, it propably would be better implement background notifications in rust.
@C0ffeeCode it sounds you do not use VAPID WebPush Notification for this insteat: https://support.mozilla.org/en-US/kb/push-notifications-firefox
Like hydrogen-web it does: https://github.com/vector-im/hydrogen-web
By its description it doesn't sound like there's an "instead" due to its severe limitations.
Looks like it'll become much more useful "soon": https://webkit.org/blog/12945/meet-web-push/ https://firt.dev/ios-15.4b#web-push-notifications-on-ios%EF%BC%8Dwith-a-catch
Now that we're using Vite, we can use Vite PWA. https://vite-pwa-org.netlify.app/frameworks/react.html
web notifs nearly work today on iOS Safari 16.4, except there seems to be a bug in the notif system, where it doesn't think it does have perms?
I have #1231 up to fix the rendering issue, though since there's not a service worker, that PR doesn't fix iOS notifications
https://elk.zone is a good example of a PWA w/ web push on iOS: https://github.com/elk-zone/elk/blob/e9740fe693cfec761cce69d47d25dc98825d0ce2/service-worker/web-push-notifications.ts#L4
https://developer.apple.com/documentation/usernotifications/sending_web_push_notifications_in_web_apps_safari_and_other_browsers
I would love to have proper iOS notification support, that seems to be the only thing I'm missing
This is a low hanging fruit if we just want to enable installation, basically just add a web manifest, right?
Features like offline support & web push integration would be great, but yeah just getting the manifest and a dummy service worker in so installs are possible would be great. ( @filipesmedeiros simply adding the manifest isn't enough, browsers will refuse to install the PWA if there is no service worker present )
@MulverineX that sounds right. Still easy just to enable installation, imo eheh
Features like offline support & web push integration would be great, but yeah just getting the manifest and a dummy service worker in so installs are possible would be great. ( @filipesmedeiros simply adding the manifest isn't enough, browsers will refuse to install the PWA if there is no service worker present )
Now that we're using Vite, we can use Vite PWA. https://vite-pwa-org.netlify.app/frameworks/react.html
Vite PWA can take care of all of this for us. I'm not familiar with React, otherwise I'd submit a PR to implement it.