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

Bundle using Webpack #4692

Merged
merged 50 commits into from
Aug 1, 2018
Merged

Bundle using Webpack #4692

merged 50 commits into from
Aug 1, 2018

Conversation

OmgImAlexis
Copy link
Collaborator

I know this is a crappy single commit with loads of changes but I just wanted feedback before I continue.

This should work out of the box in the dark theme. Do not run any gulp commands, etc. or it'll break as I've had to manually change a few things after the sync.

@sharkykh
Copy link
Contributor

sharkykh commented Jul 27, 2018

gulp sync should work correctly now, don't know about gulp build.

In the meantime we can use:

cd themes-default/slim
yarn webpack && yarn gulp sync

@sharkykh sharkykh added this to the 0.2.8 milestone Jul 27, 2018
- api.js
- router.js
- store.js

- axios
- http-vue-loader
- vue
- vue-async-computed
- vue-in-viewport-mixin
- vue-js-toggle-button
- vue-meta
- vue-native-websocket
- vue-router
- vuex
@OmgImAlexis
Copy link
Collaborator Author

@sharkykh can we try and get this merged today?

@@ -4,10 +4,10 @@
"name": "slim",
"author": "p0psicles",
"scripts": {
"gulp": "gulp",
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't need this as yarn gulp is the same even if you don't have this.

@sharkykh sharkykh changed the title try webpack Bundle using Webpack Aug 1, 2018
@OmgImAlexis OmgImAlexis merged commit 6965b93 into develop Aug 1, 2018
@OmgImAlexis OmgImAlexis deleted the feature/webpack branch August 1, 2018 12:13
@p0psicles
Copy link
Contributor

Please don't forget to update the WIKI.
It get's overlooked every time. And after the PR is merged, nobody seems to have time to update it.

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

Successfully merging this pull request may close these issues.

None yet

3 participants