Fix injection issues

[QWp6t]

This should hopefully fix issues with browser-sync not refreshing or injecting when changes are made to template files.

Note that we are no longer passing the copy glob to browser-sync. We are only passing php files to it. Webpack handles everything else.

Still a couple of issues to address:

• (upstream) Problems with SSL
• Compatibility issue with Tailor (wontfix? - seems to be a non-issue to me, just briefly shows an error message but works fine thereafter)

[derkjn]

Tested this this morning, all seems to be working fine on my end as far as I can tell!
Thanks for the fix @QWp6t!

[apintocr]

I confirm this is working after the latest commit 419863e
However the JS might need to be reloaded after as, for example, select2 stops working after the HTML injection takes place.

[TrayWeb]

Hi Guys,

I still have the same issue on a fresh install of sage 9 beta 3.
CSS files work well but not the blades.

Does anyone still have the same issue ?

[apintocr]

This commit solved the issue on my previous project, however a fresh install I still have the same issue.

[QWp6t]

What version of node are you using?

[TrayWeb]

Thanks for the reply:

v7.8.0

---

[QWp6t]

I think they removed or renamed the environment variable for 7.x, but I haven't had a chance to look up how it should be done now.

The last I had read about it was in this discussion: nodejs/node#5258

[TrayWeb]

Hello Guys,
Fixed the issue by changing the watch part on my config.json:

  "watch": [
    "resources/views/partials/*.blade.php",
    "resources/views/layouts/*.blade.php",
    "resources/views/*.blade.php"
  ],

[apintocr]

@TrayWeb not working for me :(

[TrayWeb]

Can you please send your config.json file and also the structure of your files ?
Does it work for the css ?

[apintocr]

@TrayWeb it's a clean sage install using the latest commits from the repository.

Anyway, here's my config.json

{
  "entry": {
    "main": [
      "./scripts/main.js",
      "./styles/main.scss"
    ],
    "customizer": [
      "./scripts/customizer.js"
    ]
  },
  "publicPath": "/app/themes/themename",
  "devUrl": "https://site.dev",
  "proxyUrl": "https://localhost:3000",
  "cacheBusting": "[name]_[hash:8]",
  "watch": [
    "{app,resources/views}/**/*.php"
  ],
  "browsers": [
    "last 2 versions",
    "android 4",
    "opera 12"
  ]
}

I've also tried with your code, no change.

[apintocr]

Funny enough I have another project, also with sage (beta-3 here), that has EXACTLY the same webpack files and configuration, it's using the same Trellis vagrant box and it works...

(this is not a clean sage project, it's an already fully developed site)

[waylay]

@TrayWeb Thanks! Worked for me! This is what I used:

  "watch": [
    "app/*.php", 
    "resources/views/**/*.php", 
    "resources/views/*.php"
  ]

[coffeeneed]

Thanks, @TrayWeb
I also experienced this (latest commit) problem and can confirm that this worked;

Changing from:

  "watch": [
    "{app,resources/views}/**/*.php"
  ]

too:

  "watch": [
    "resources/views/partials/*.blade.php",
    "resources/views/layouts/*.blade.php",
    "resources/views/*.blade.php"
  ]

Any chance a fix for this will be commited to master any time soon?

[v3nt]

was this ever fixed? Started a new project and still not watching blade files.