TylerBarnes / gatsby-plugin-transition-link

A link component for page transitions in gatsby
537 stars 70 forks source link

Doesn't work with Sanity Gatsby setup #134

Open patrickbjohnson opened 5 years ago

patrickbjohnson commented 5 years ago

I'd add more but honestly, installing and including in gatsby-config is enough to break the build locally.

Seems like it looks for react-dom. After installing that, it fails due to not finding @reach/router. Then after that it errors at No PostCSS Config.

TylerBarnes commented 5 years ago

Sounds like there might be something else going on with your project. Those packages are required by Gatsby so they should be available. Are you able to share your repo so I can take a look?

patrickbjohnson commented 5 years ago

I can in the morning!

Patrick Johnson Front-End Developer 407.460.4813 @patrickbjohnson ( https://twitter.com/patrickbjohnson ) www.pbj.me ( http://www.pbj.me/ )

On Thu, Sep 26 2019 at 9:29 PM, < notifications@github.com > wrote:

Sounds like there might be something else going on with your project. Those packages are required by Gatsby so they should be available. Are you able to share your repo so I can take a look?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub ( https://github.com/TylerBarnes/gatsby-plugin-transition-link/issues/134?email_source=notifications&email_token=AAOZMSM76OUDTSJF5O4AUSTQLVOWNA5CNFSM4I3AQ2YKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7XNGKA#issuecomment-535745320 ) , or mute the thread ( https://github.com/notifications/unsubscribe-auth/AAOZMSJ235BMK4RWPT3EUCTQLVOWNANCNFSM4I3AQ2YA ).

patrickbjohnson commented 5 years ago

@tyleratbare adding you now as a collaborator.

TylerBarnes commented 5 years ago

Hey @patrickbjohnson I pulled down your project, installed the dependencies, and ran gatsby develop. I think there are general problems with the project you should fix before trying to install transition link. It's not yet installed but I'm getting a bunch of errors about missing packages.

Closing this issue as it appears at least some of the problems you're experiencing are occurring before transition link is installed. Feel free to keep asking questions here though, I don't mind helping out.

I needed to install lodash.orderby and axios-jsonp in order to get gatsby develop to run but then I'm getting a GraphQL error in the browser

GraphQL Error Encountered 1 error(s):
- Unknown field 'allCampaign' on type 'Query'.

      file: /Users/tylerbarnes/Documents/gatsby/sanity-gatsby-portfolio/web/src/pages/newsletter.js
TylerBarnes commented 5 years ago

Actually, let's keep this open until we're sure it works with the Sanity plugin.

patrickbjohnson commented 5 years ago

@TylerBarnes Hey. Can you run me thru the steps you took to get those errors? I tried to recreate your the missing packages but couldn't.

spacedawwwg commented 5 years ago

I'm having this issue too.

Error: No PostCSS Config found in: /my-project/node_modules/gatsby-plugin-transition-link at config.search.then (/my-project/node_modules/postcss-load-config/src/index.js:91:15)

Also using Sanity

patrickbjohnson commented 5 years ago

I've actually gotten this to work within Sanity.

My original inclination that I should be adding this plugin at the root of the project. Most template sanity projects are mono-repos so they use lerna to manage the studio portion of sanity and the web portion of the front-end.

When I installed this plugin into the web portion of the mono-repo I had no issues getting it to run.

TylerBarnes commented 4 years ago

@patrickbjohnson good to know! thanks for sharing. @spacedawwwg did you have a chance to try installing there?