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

grunt dev admin livereload #590

Merged
merged 1 commit into from
Mar 30, 2017
Merged

Conversation

kevinansfield
Copy link
Member

@kevinansfield kevinansfield commented Mar 16, 2017

refs TryGhost/Ghost#8161, requires TryGhost/Ghost#8176

  • updates the ember:watch grunt task to use ember serve, passing in the required --live-reload-base-url parameter
  • removes the unused ember-cli-content-security-policy dependency to remove console log noise from x-domain warnings

With this and the required Ghost PR, when using grunt dev the admin screen will refresh any time a file is changed. It will also allow tests to be run simultaneously by visiting http://localhost:4200/tests

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 516d75a on kevinansfield:livereload into ** on TryGhost:master**.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 75.012% when pulling 4df06a1 on kevinansfield:livereload into 3b27fe5 on TryGhost:master.

@kevinansfield kevinansfield changed the title [WIP] ✨ grunt dev admin livereload grunt dev admin livereload Mar 28, 2017
@coveralls
Copy link

Coverage Status

Coverage remained the same at 75.012% when pulling 753d229 on kevinansfield:livereload into 3b27fe5 on TryGhost:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 75.012% when pulling 28a386d on kevinansfield:livereload into 3b27fe5 on TryGhost:master.

refs TryGhost/Ghost#8161, requires tbd
- updates the `ember:watch` grunt task to use `ember serve`, passing in the required `--live-reload-base-url` parameter
  - forces live reload port to `4201` for now as there's a [bug on touchbar macs](ember-cli/ember-cli#6513)
- removes the unused `ember-cli-content-security-policy` dependency to remove console log noise from x-domain warnings

With this and the required Ghost PR, when using `grunt dev` the admin screen will refresh any time a file is changed. It will also allow tests to be run simultaneously by visiting http://localhost:4200/tests
@coveralls
Copy link

Coverage Status

Coverage remained the same at 74.994% when pulling 148375d on kevinansfield:livereload into 5203681 on TryGhost:master.

kevinansfield added a commit to kevinansfield/Ghost that referenced this pull request Mar 30, 2017
refs TryGhost#8161, requires TryGhost/Admin#590
- adds a development-only route to the admin app that redirects to ember-cli's livereload script
- updates Gruntfile `watch` task to pass the `live-reload-base-url` param with subdirectory support
- updates Gruntfile `bgShell:client` task to filter potentially confusing output from `ember serve`
  - removes `Livereload server on http://localhost:49153`
  - removes `Serving on http://localhost:4200/`

With this and the required Ghost-Admin PR, when using `grunt dev` the admin screen will refresh any time a file is changed. It will also allow client tests to be run simultaneously by visiting http://localhost:4200/tests
@ErisDS ErisDS merged commit f05f31f into TryGhost:master Mar 30, 2017
ErisDS pushed a commit to TryGhost/Ghost that referenced this pull request Mar 30, 2017
refs #8161, requires TryGhost/Admin#590
- adds a development-only route to the admin app that redirects to ember-cli's livereload script
- updates Gruntfile `watch` task to pass the `live-reload-base-url` param with subdirectory support
- updates Gruntfile `bgShell:client` task to filter potentially confusing output from `ember serve`
  - removes `Livereload server on http://localhost:49153`
  - removes `Serving on http://localhost:4200/`

With this and the required Ghost-Admin PR, when using `grunt dev` the admin screen will refresh any time a file is changed. It will also allow client tests to be run simultaneously by visiting http://localhost:4200/tests
@kevinansfield kevinansfield deleted the livereload branch June 4, 2018 10:01
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