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

fix(examples): GraphQL reference codesandbox issue #10807

Merged
merged 2 commits into from
Jan 4, 2019
Merged

Conversation

LekoArts
Copy link
Contributor

@LekoArts LekoArts commented Jan 3, 2019

Codesandbox apparently fails on that

image

Copy link
Contributor

@DSchau DSchau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could probably just refactor the blog template slightly--but this works just as well.

Nice; thank you!

@DSchau
Copy link
Contributor

DSchau commented Jan 3, 2019

Actually... this should be fixed by a4621c6 and the PR #10220 correct?

cc @pieh

@pieh
Copy link
Contributor

pieh commented Jan 3, 2019

Actually... this should be fixed by a4621c6 and the PR #10220 correct?

Yeah, I hit this when I was using blog starter after deleting .eslintrc file there, so I added that to common eslint config

@LekoArts
Copy link
Contributor Author

LekoArts commented Jan 3, 2019

Codesandbox doesn't have access to that file (I think)

@pieh
Copy link
Contributor

pieh commented Jan 3, 2019

It wasn't published yet I think

@LekoArts
Copy link
Contributor Author

LekoArts commented Jan 3, 2019

Ah, I see. However I think Codesandbox will only restart the dev server (instance) if something changed on the repo. So if we don't change the PATH_PREFIX we'd need to change something else to trigger a new run.

@DSchau
Copy link
Contributor

DSchau commented Jan 3, 2019

However I think Codesandbox will only restart the dev server (instance) if something changed on the repo

We'd need to upgrade to the version of gatsby that contains this change--so maybe we do that in this PR?

@LekoArts LekoArts changed the title fix(examples): Remove PATH_PREFIX fix(examples): GraphQL reference codesandbox issue Jan 4, 2019
Copy link
Contributor

@m-allanson m-allanson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💪 Thanks!

@m-allanson m-allanson merged commit 159d008 into master Jan 4, 2019
@m-allanson m-allanson deleted the LekoArts-patch-1 branch January 4, 2019 11:00
@LekoArts
Copy link
Contributor Author

LekoArts commented Jan 4, 2019

Didn't work out as expected, .83 didn't have the fix in it.

@DSchau
Copy link
Contributor

DSchau commented Jan 4, 2019

@LekoArts I'm not sure the change has been published yet.

The eslint-config.js does not contain the additional global.

@pieh
Copy link
Contributor

pieh commented Jan 4, 2019

I just published .84 with the change

m-allanson added a commit that referenced this pull request Jan 4, 2019
DSchau pushed a commit that referenced this pull request Jan 4, 2019
## Description

A version of Gatsby with the correct fix has been published, this is the same as #10807
gpetrioli pushed a commit to gpetrioli/gatsby that referenced this pull request Jan 22, 2019
* Remove PATH_PREFIX from graphql-reference-example

* update package instead
gpetrioli pushed a commit to gpetrioli/gatsby that referenced this pull request Jan 22, 2019
## Description

A version of Gatsby with the correct fix has been published, this is the same as gatsbyjs#10807
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

4 participants