Skip to content
This repository has been archived by the owner on Nov 15, 2021. It is now read-only.

use vue-cli 3.0 way when it's ready #2

Closed
DrSensor opened this issue Jan 10, 2018 · 4 comments
Closed

use vue-cli 3.0 way when it's ready #2

DrSensor opened this issue Jan 10, 2018 · 4 comments

Comments

@DrSensor
Copy link
Owner

DrSensor commented Jan 10, 2018

This issue for anticipating vue-cli 3.0.

Currently, this template using poi because vue build was removed. Poi is great for scaffolding project and hide the configuration project in prebuilt poi-presets. However, it's also not really copas friendly because it uses webpack-chain format for extending webpack configuration while many webpack-loader and storybook-addon in their README use curly brackets format and that make some user confused how to add that addon/loader. To make it copy-paste friendly, need to introduce config.merge(curlyBracketConfig) in the .storybook/poi.config.js.

Take for example storybook-readme, if this template use webpack config of storybook then the user will know where to put storybookBaseConfig to apply that addon.

need to keep on track poi-preset-storybook with storybook default config. Create commit change bot maybe...

@egoist
Copy link

egoist commented Jan 18, 2018

Well we can add markdown-loader to the storybook preset if that's supported by storybook.

btw vue-cli 3.0 uses webpack-chain too.

@DrSensor
Copy link
Owner Author

Interesting. Is there any benefit using webpack-chain? I find it hard using webpack-chain because many webpack config example use curly-bracket format.

@egoist
Copy link

egoist commented Jan 18, 2018

This introduction explains it pretty well https://github.com/mozilla-neutrino/webpack-chain#introduction

@DrSensor DrSensor changed the title Remove poi and use vue-cli 3.0 way when it's ready use vue-cli 3.0 way when it's ready Jan 19, 2018
DrSensor added a commit that referenced this issue Jan 23, 2018
- Relate #2, reason for removing poi
- Relate #1, open possibility to deploy to gh-pages
DrSensor added a commit that referenced this issue Jan 24, 2018
- Relate #2, reason for removing poi
- Relate #1, open possibility to deploy to gh-pages
DrSensor added a commit that referenced this issue Jan 24, 2018
- Relate #2, reason for removing poi
- Relate #1, open possibility to deploy to gh-pages
@DrSensor DrSensor closed this as completed Feb 5, 2018
@DrSensor
Copy link
Owner Author

DrSensor commented Feb 5, 2018

Switch to #15

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

No branches or pull requests

2 participants