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

Node relations #1

Merged
merged 10 commits into from
Sep 7, 2018
Merged

Conversation

isabellachen
Copy link
Contributor

Update this plugin so now it is possible to query venue__NODE when querying events to get the venue.

For example, with a query like this:

{
  allEventbriteEvents {
    edges {
      node {
        name {
          text
        }
        venue__NODE {
          name
          address {
            address_1
            address_2
            city
          }
        }
      }
    }
  }
}

@isabellachen
Copy link
Contributor Author

Also, I'm unsure of the best way to go about adding or changing node relations since nodes don't get recreated unless the node's internal.contentDigest. A work around is to de-create the nodes and re-create them. You can do this by commenting out the line that uses createNode(), recompile with gatsby develop and uncomment the line and recompile but there's probably a better way...

@sarasate
Copy link

sarasate commented Sep 2, 2018

I'll have a look at this next week..

Copy link

@sarasate sarasate left a comment

Choose a reason for hiding this comment

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

Thanks for the discovery. I added the fix to the repo manually for now. The PR is not working as intended for me so can't merge. I'll have a closer look at this soon..

src/gatsby-node.js Show resolved Hide resolved
@isabellachen
Copy link
Contributor Author

isabellachen commented Sep 5, 2018 via email

@sarasate sarasate merged commit a2e562a into GatsbyCentral:master Sep 7, 2018
@sarasate
Copy link

sarasate commented Sep 7, 2018

🎉 This PR is included in version 1.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

2 participants