Skip to content
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

Build is failing #44

Closed
AustinGreen opened this issue Feb 26, 2018 · 9 comments
Closed

Build is failing #44

AustinGreen opened this issue Feb 26, 2018 · 9 comments

Comments

@AustinGreen
Copy link
Contributor

Hey @erquhart, any idea why the build is failing? It's failing on gatsby-netlify-cms.netlify.com and I was able to replicate it locally. Below is the error I'm getting:

error Generating JavaScript bundles failed
  Error: app-57675578fd6eed3803db.js from UglifyJs
  SyntaxError: Unexpected character '`' [./~/gatsby-plugin-netlify-cms/gatsby-browser.js:10,0]
@erquhart
Copy link
Contributor

Just waiting for a release on the Gatsby repo.

@erquhart
Copy link
Contributor

The fix was in gatsbyjs/gatsby#4233, which is now released, just pushed a lockfile update and restarted the build.

@erquhart
Copy link
Contributor

erquhart commented Feb 26, 2018

Fixed, but looks like something from the recent changes broke a query.

5:10:59 PM: success Building production JavaScript bundles — 52.268 s
5:11:04 PM: error Building static HTML for pages failed
5:11:04 PM: See our docs page on debugging HTML builds for help https://goo.gl/yL9lND
5:11:04 PM:   36 |   return (
5:11:04 PM:   37 |     <BlogPostTemplate
5:11:04 PM: > 38 |       content={post.html}
5:11:04 PM:      |                     ^
5:11:04 PM:   39 |       contentComponent={HTMLContent}
5:11:04 PM:   40 |       description={post.frontmatter.description}
5:11:04 PM:   41 |       helmet={<Helmet title={`Blog | ${post.frontmatter.title}`} />}
5:11:04 PM: 
5:11:04 PM:   WebpackError: Cannot read property 'html' of null
5:11:04 PM:   
5:11:04 PM:   - blog-post.js?ab37:38 exports.default
5:11:04 PM:     src/templates/blog-post.js?ab37:38:21

@AustinGreen
Copy link
Contributor Author

Hmm, looks like #41 removed necessary graphql fields. Seems like you caught it here 2259ee4#r170452856

I can add these back and make a PR

@rdela
Copy link
Contributor

rdela commented Feb 26, 2018

☝️This concurs with my assessment messing around with https://github.com/rdela/voidcluster

@AustinGreen
Copy link
Contributor Author

Create PR #45. I'm waiting for the build to pass and then I'll merge.

@erquhart
Copy link
Contributor

Fixed in #46.

@rdela
Copy link
Contributor

rdela commented Feb 27, 2018

Yeah I was wrong earlier fix in in #46 thanks @erquhart @AustinGreen
@KyleAMathews!
#46 (comment)

Removing passing in path surfaced a bug in the data where one of the paths was missing its forward slash. Gatsby adds to paths if you don't add it but because of the discrepancy, the query now failed because the path in the page object was different than the path in the markdown file.

rdela/voidcluster@0bf263a

@aboutaaron
Copy link

I fixed this issue by upgrading Gatsby to the latest version (currently 1.9.243)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants