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

Updated dependencies, Refactored service to provider, optimized npm cmds #86

Closed
wants to merge 2 commits into from

Conversation

jbreuer95
Copy link

Addresses #53 #66 and renders #74 not needed

What I did:

  • Removed the nuxtbuild command and replaced this with the native nuxt build command with the custom conf file.
  • Made a Service Provider of the Service with the best practices from the Adonis documentation
  • Updated all the dependencies to that of the official full-stack template
  • Replaced nodemon with the native adonis serve --dev command

This is quite a big overhaul so it should definitely be carefully checked. I will write some checks next week if I have the time, for now, I really wanna go to sleep :)

@jbreuer95
Copy link
Author

@atinux Nice talk at Vueconf-Amsterdam! Any chance you can take a look at my PR?

@alanaasmaa
Copy link

Hi, i have tested that and it seems to work fine when working on backend. But when i use frontend i have to manually reload the page for Nuxt to re render. Sometimes it works though.

I think manual refresh is needed when i edit something that is more close to core ex plugins.

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

Successfully merging this pull request may close these issues.

3 participants