Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feature request: add service-worker support #647

Closed
alexey2baranov opened this issue Apr 2, 2017 · 6 comments
Closed

feature request: add service-worker support #647

alexey2baranov opened this issue Apr 2, 2017 · 6 comments

Comments

@alexey2baranov
Copy link

Hello!
it would be very helpfully if your template have another one ask when initiated "Service worker support (Y/n)?"

And if I set "Yes" some steps should be performed:

  1. add serviceworker-webpack-plugin to package.json dependency
  2. /test/unit/index.js should exclude src/sw.js
  3. karma.conf.js should configure to allow register sw.js from test

is it possible ? thanks!

@LinusBorg
Copy link
Contributor

Thanks for this suggestion.

However, we currently don't want to make the template heavier by adding more questions in the initial setup. Also, Service Workers would rather be a topic for a special "PWA" template.

We will overhaul this template soon anyway, hopefully in the coming weeks, so I will close this request for now. This feature should be added in another template.

@maoberlehner
Copy link

maoberlehner commented Feb 12, 2018

Hey @LinusBorg,

I know there is a separate PWA template https://github.com/vuejs-templates/pwa but I don't think this is the best way to handle this feature. Because, although the differences between this and the PWA template are / were very minor, the PWA template is constantly out of sync with this template, which is annoying. I considered to contribute my time to keep the PWA template in sync, but I don't think it would be the best use of my time, to be honest.

I'd like to work on a pull request which adds (opt-in!) PWA support to this template, similar to how create-react-app handles this in its upcoming release (facebook/create-react-app#3817).

Would you consider accepting such a pull request, or should I work on the PWA template instead?

@LinusBorg
Copy link
Contributor

Hi @maoberlehner,

Thanks for contributing. However I don't think that we should add PWA support to this template - primarily because vue-cli 3 will come with (opt-in) support for it anyway.

So my general strategy for this template would be to not add any more bigger features, especially when they are covered by the new cli.

I will make an exception for typescript support mainly because I wanted to merge the OR for this for about half a year and didn't get around to it...

@maoberlehner
Copy link

Thanks for your quick answer, didn't know about the big changes coming with vue-cli 3, I guess than It's fine :)

@FossPrime
Copy link

@maoberlehner Some docs on how to retrofit it would be welcome... did you figure it out?

@maoberlehner
Copy link

@rayfoss no, I'm using Vue CLI for everything now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants