Serubin / pulse-sms-web

The official web app for Pulse SMS - built on Vue.js.

Home Page:https://pulsesms.app

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Update Vue

Serubin opened this issue · comments

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:

  • Initial conversion/update (89302af)
  • Fix service worker (5a02927)
  • Fix favicon (it was still green on initial testing)? (84ab8b8)
  • Verify deployment works

See feature/update-vue branch.

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.

Service workers are functional and favicons/icons are perfect.

Working well at pulse-beta.serubin.net

Done and deployed 👍