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

General improvements #21

Merged
merged 27 commits into from
Mar 19, 2017
Merged

General improvements #21

merged 27 commits into from
Mar 19, 2017

Conversation

euvl
Copy link
Contributor

@euvl euvl commented Mar 13, 2017

Started using sass (#19), added config file with flags to turn on/off fixedLayout, some refactoring. Also replaced VueResource with Axios (as Vue.js do not favour VueResource anymore)

@koss-shtukert
Copy link

Great Job!

@euvl
Copy link
Contributor Author

euvl commented Mar 13, 2017

Some updates might be arguable though, e.g. i've removed private google analytics token (that might have been there for a reason) and replaced VueResource, which is a bit obsolete. Will be happy to discuss any questions with maintainers.

  • Also planning to sort out /static folder as most of the dependencies are already there in node_modules

@misterGF
Copy link
Owner

Great PR! Thank you. Will give it a closer look this week.

@euvl
Copy link
Contributor Author

euvl commented Mar 14, 2017

Perfect, thanks @misterGF! 😄

“Note that nested paths that start with / will be treated as a root
path. This allows you to leverage the component nesting without having
to use a nested URL.” (c) docs

https://router.vuejs.org/en/essentials/nested-routes.html
@shellscape
Copy link

👍 from me

@misterGF misterGF merged commit 08212c1 into misterGF:master Mar 19, 2017
@misterGF
Copy link
Owner

Updated deployed to demo site!

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

Successfully merging this pull request may close these issues.

None yet

4 participants