Skip to content

Conversation

@ckj
Copy link
Contributor

@ckj ckj commented Jul 24, 2018

This PR:

  • Migrates to Gatsby 2.0
  • Fixes an issue where firebase.auth() would cause the build to fail
  • Formats the code via npm run format
@rwieruch
Copy link
Member

rwieruch commented Jul 25, 2018

Oh wow, this is awesome!

If I remember correctly, I introduced gatsby-next because of this issue with React's context API: #1 I guess this stays fixed then. Did you experience any bugs there? Otherwise I merge this. Thanks for your efforts! Helps a lot! 💯

@ckj
Copy link
Contributor Author

ckj commented Jul 25, 2018

If I remember correctly, I introduced gatsby-next because of this issue with React's context API: #1 I guess this stays fixed then.

I was told in this issue that it was no longer necessary in v2. I personally didn't experience any issues but it might something worth confirming on your end before merging.

Happy to help!

@rwieruch
Copy link
Member

Thanks again!

@rwieruch rwieruch merged commit e6ccd18 into the-road-to-react-with-firebase:master Jul 25, 2018
"version": "1.0.0",
"author": "Robin Wieruch <wrobin@gmx.net>",
"dependencies": {
"bluebird": "^3.5.1",
Copy link
Member

Choose a reason for hiding this comment

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

@ckj do you know where Bluebird is used in the application?

@rwieruch rwieruch mentioned this pull request Aug 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants