Skip to content

angular-cli-builders:custom-webpack-server not work #1

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

Closed
AlexDaSoul opened this issue Jul 3, 2018 · 3 comments
Closed

angular-cli-builders:custom-webpack-server not work #1

AlexDaSoul opened this issue Jul 3, 2018 · 3 comments

Comments

@AlexDaSoul
Copy link

Hi! angular-cli-builders:custom-webpack-server finished with errors

Schema validation failed with the following errors:
Data path "" should NOT have additional properties(webpackConfigPath).

if remove webpackConfigPath then the next error is next property and more and more and more
when remove all except for main. ng serve finished as ng buld, not start dev-server

@arjanburggraaf
Copy link

arjanburggraaf commented Jul 3, 2018

Hi @meltedspark, thank you for providing this small but very useful builder!

I ran into the same issue.
I noticed that 'webpackConfigPath' is missing in: src/custom-webpack/server/schema.json.
But 'webpackConfigPath' is present in src/custom-webpack/browser/schema.json.

@just-jeb
Copy link
Owner

just-jeb commented Jul 3, 2018

Hi, thanks for reporting this issue.

@arjanburggraaf indeed webpackConfigPath was missing from the schema, it's fixed in version 1.0.3. Thank you for reporting the root cause.

@AlexDaSoul the reason it complains about such properties as index, styles, scripts etc. is that these properties are not supported in the original @angular-devkit/build-angular:server and, as README states, this builder is an extension that adds one more entry webpackConfigPath.
The README was confusing though, as it had these properties in the sample angular.json for server. This is fixed now.
Check out the full server schema and properties description.
Also, you may find this and this useful for building universal app.

@just-jeb
Copy link
Owner

just-jeb commented Jul 3, 2018

I'm reopening this issue as a reminder for myself to write a test but generally it is fixed in build 1.0.3

@just-jeb just-jeb reopened this Jul 3, 2018
@just-jeb just-jeb mentioned this issue Jul 6, 2018
just-jeb pushed a commit that referenced this issue Aug 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants