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

WIP: fix: fixes regression with pathPrefix and gatsby-link #1579

Closed
wants to merge 1 commit into from

Conversation

DSchau
Copy link
Contributor

@DSchau DSchau commented Jul 20, 2017

Note: this is not done! Want to get the tests actually working. Unsure why the globals aren't setting, and why I need to use Jest globals :/

Also similar to #1574. We should probably consolidate our efforts

Note: this is not done! Want to get the tests actually working
@KyleAMathews
Copy link
Contributor

Deploy preview failed.

Built with commit 26c2d78

https://app.netlify.com/sites/using-glamor/deploys/5970dac87960b15b567a2c7a

@KyleAMathews
Copy link
Contributor

Deploy preview failed.

Built with commit 26c2d78

https://app.netlify.com/sites/using-remark/deploys/5970dac87960b15b567a2c83

@gatsbybot
Copy link
Collaborator

Deploy preview ready!

Built with commit 26c2d78

https://deploy-preview-1579--using-drupal.netlify.com

@KyleAMathews
Copy link
Contributor

Deploy preview failed.

Built with commit 26c2d78

https://app.netlify.com/sites/gatsbygram/deploys/5970dac77960b15b567a2c68

@KyleAMathews
Copy link
Contributor

Deploy preview failed.

Built with commit 26c2d78

https://app.netlify.com/sites/gatsbyjs/deploys/5970dac77960b15b567a2c65

@KyleAMathews
Copy link
Contributor

Looks like the tests are failing due to the glamor issue I'm fixing in #1580

@DSchau
Copy link
Contributor Author

DSchau commented Jul 20, 2017

@KyleAMathews yeah I wasn't so concerned with those, and figured they'd be fixed at some point.

Just unsure why I can't set globals like I am for the DefinePlugin replaced values :/ It'd be nice to have some tests so we can catch a regression in the future

@KyleAMathews
Copy link
Contributor

Looks like you're setting them right... https://stackoverflow.com/a/40460395

@KyleAMathews
Copy link
Contributor

Merged #1574 and making a release. Would love to get your tests in soon!

@DSchau DSchau closed this Jul 20, 2017
@DSchau DSchau deleted the link-fix branch July 20, 2017 17:55
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

Successfully merging this pull request may close these issues.

None yet

3 participants