search

Serubin / pulse-sms-web

The official web app for Pulse SMS - built on Vue.js.
https://pulsesms.app
Other
109 stars 44 forks source link

Update Vue #82

Closed Serubin closed 5 years ago

Serubin commented 5 years ago

Update Vuejs to latest version. Newer versions of Vue provide an improved build system and better upgrading and integration of tools.

This provides a nice launching point to cleaning up the code base.

Current progress:

See feature/update-vue branch.

Serubin commented 5 years ago

Service-worker loader is not currently doing everything it's supposed too.

Error on start:

uncaught exception: Object
The Notification permission may only be requested from inside a short running user-generated event handler. login
Failed to register/update a ServiceWorker for scope ‘https://pulse.serubin.net/’: Load failed with status 404 for script ‘https://pulse.serubin.net/service-worker.js’.
TypeError: ServiceWorker script at https://pulse.serubin.net/service-worker.js for scope https://pulse.serubin.net/ encountered an error during installation.
Serubin commented 5 years ago

Service workers are functional and favicons/icons are perfect.

Serubin commented 5 years ago

Working well at pulse-beta.serubin.net

klinker24 commented 5 years ago

Done and deployed 👍