-
Notifications
You must be signed in to change notification settings - Fork 12k
add --inline-client command option #3381
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
Conversation
name: 'inline-client', | ||
type: String, | ||
aliases: ['ic'], | ||
description: '(Defaults to "http://host:port")' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add proper description on what it does please.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added a description, any suggestions?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Websocket client URL for Webpack inline mode (Defaults to "http://host:port")
sounds reasonable
Also please rebase your code with the latest master. |
… livereload in webpack-dev-server
@elclanrs thx, this fixed my issue now i can do this should be merged asap, until then i use npm link and keep merging in the latest master |
@hansl Hi, is there any reason why this hasnt been merged? I would happily rebase it with the latest master and resolve the conflict -the run method has been separated to serve.run.ts-, if you say u can merge it. If you can't merge it, can you please tell why? |
Looks good, just need to rebase. |
@hansl as per #3873 (comment), #3952 appears to supersede this PR in a more elegant way, by fixing options that we already have but are not working currently. |
Closed in favor of #3952 |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
This is a fix for issue #3361. This PR adds a new
--inline-client
option to customize Webpack's inline client URL that makes live reloading work. This is useful when your are using Vagrant or Docker and the client and host names and/or ports may differ. For example, inside Vagrant I may runng serve
on the default 4200 port and reverse proxy to port 80 which I expose in my host as port 1234. But angular-cli prints the wrong URL:Producing this error:
With this new option I can change it:
The above will use the right URL: