-
Notifications
You must be signed in to change notification settings - Fork 10.3k
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
GraphCMS source plugin and usage example #2445
Conversation
Deploy preview failed. Built with commit 74fb6cc https://app.netlify.com/sites/using-glamor/deploys/5a06c3f38198765905f70eec |
Deploy preview ready! Built with commit 9dcd5fe |
…itations and TODOs
- removed unused code - added logic to fail processing if GraphQL query parameter is not passed in configuration
@KyleAMathews Anything holding this up? |
Issue number 2 in PR #1867 is holding it back. The data isn't found by graphql at build time, despite all the data being pulled in. It might be a simple change to the way There is chat going on in the |
Preventing fields being named after 1 of the faulty keywords
So I built the plugin in this working tree (still no |
|
Hmm |
Ok I am still stumbling around in the dark with yarn but example code seems to have survived Going to add recommended |
Ok @KyleAMathews think this is ready for your eyes, AWS CodeBuild succeeded at least |
Some more changes to bring this more in line and conform with existing source plugins and ditch cruft/hopefully appease build servers. All other source plugins… - do not have a .babelrc - have babel-runtime in dependencies not devDependencies - have "prepublish": "cross-env NODE_ENV=production npm run build" in scripts so experimentally trying these chages
By gatsby-source-graphcms-plugin devDep combo in that last commit msg I mean babel-plugin-transform-object-rest-spread (in) devDep combo |
Confused about what Babel needs to cover all cases and what is extraneous and unsure whether we have everything here to roll this out, but this is everything I could think to do tonight without more insight, feel free to modify anything I have done here so far |
AWS CodeBuild succeeded for second or third time running so not sure whether any of the last few commits or presence or content of .babelrc make any difference to builds on my system. Earlier this .babelrc resolved a build issue for me but may be moot now or need to be swapped for recommended package format style .babelrc: https://github.com/gatsbyjs/gatsby/blob/master/plop-templates/package/.babelrc.hbs Or deleted?bf47613
Travis seems stuck and netlify build is failing, but think that may be unsatisfied dep for 0.0.1 of this plugin in example. Curious what are real issues and need attention and what will resolve once published. |
I guess I need to read up on Lerna hoisting to figure out Babel dev/production logic in Gatsby |
@KyleAMathews what do you need me to do here? Fix something specific? Merge current master into this branch? Get @Redmega to reopen PR #1867? @hmeissner or me open a new PR with clean tree based on current? We are working on new features but dev is getting splintered with https://github.com/hmeissner/gatsby-graphcms-testing being more up to date and approachable. With so many different people having now committed to the brach over 2 PRs and not knowing
|
I guess 3c5df69 was last successful build on Oct 13 #2445 (commits) |
Hey! First thanks for all the great work here! Sorry about the confusion AWS CodeBuild is causing — I'm working on a new testing tool and it's a WIP still so giving weird results. I've been thinking too lately that our experiment in putting lots of plugins in this repo is getting unwieldy — I'm reaching my limits for how much maintenance I can do. How about you put this in your own repo or perhaps a GraphQLCMS repo? A community member is working on a design for a new plugins page that'll make it really easy to browse/search plugins so it won't be that big of deal anymore which repo plugins come from. Thoughts? |
I definitely don't have a problem hosting this repo on my own github and publishing to npm separately. I can work on that after work tonight if everyone thinks we're ready to go live with |
@KyleAMathews We're going to host it in the @GraphCMS org. |
Sweet!! |
Yeah I am super pro all these things @Redmega and @KyleAMathews thank you both let’s close this please! |
Or we can wait until 0.0.1 is on npm and then add comment here before closing whatever is clever |
live under @GraphCMS org https://github.com/GraphCMS/gatsby-source-graphcms |
@corysimmons please let us know if you want to be more involved or have thoughts on alpha https://github.com/GraphCMS/gatsby-source-graphcms / hmeissner/gatsby-graphcms-testing#2 |
New repo created at https://github.com/GraphCMS/gatsby-source-graphcms and all commits for this PR pushed there. |
GraphCMS source plugin and usage example