-
Notifications
You must be signed in to change notification settings - Fork 12k
scripts path in index.html does not follow deployUrl defined in .angular-cli.json in 1.4.0 #7628
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
Comments
Same issue here. Versions:
Repro:
Running
Workaround for now is to fix the path manually, but it is a small issue during development. Additional info: |
same here. How soon do they normally fix these sort of bugs? This was an untimely release for us. |
We should fix this one ASAP. I'm investigating and will try to do an emergency release today. |
This has been released in 1.4.1. Try it out and if there's a problem, open another issue please. |
@hansl updated to 1.4.1 |
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. |
Bug Report or Feature Request (mark with an
x
)Versions.
@angular/cli: 1.4.0
node: 8.0.0
os: darwin x64
@angular/animations: 4.3.6
@angular/common: 4.3.6
@angular/compiler: 4.3.6
@angular/core: 4.3.6
@angular/forms: 4.3.6
@angular/http: 4.3.6
@angular/platform-browser: 4.3.6
@angular/platform-browser-dynamic: 4.3.6
@angular/router: 4.3.6
@angular/cli: 1.3.2
@angular/compiler-cli: 4.3.6
@angular/language-service: 4.3.6
Repro steps.
.angular-cli.json: (Note the
deployUrl
)After I run
ng build --prod
, theindex.html
intarget/my-app
:Note the scripts path is not under
my-app
, but the the scripts file is undermy-app
directory. As a result, scripts cannot be fount in browser.1.3.2 does not have this problem.
Thanks!
The text was updated successfully, but these errors were encountered: