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

v2 umbrella issue #1824

Closed
5 of 14 tasks
KyleAMathews opened this issue Aug 15, 2017 · 7 comments
Closed
5 of 14 tasks

v2 umbrella issue #1824

KyleAMathews opened this issue Aug 15, 2017 · 7 comments

Comments

@KyleAMathews
Copy link
Contributor

KyleAMathews commented Aug 15, 2017

v2 you say? But we just released a v1???

Well… yes but we're on webpack 1 and webpack 3 is out and Babel 7 is almost out etc. so there's some breaking changes we need to make to upgrade so new major release it is!

The changes should be pretty minor for most sites. Unless you're directly modifying webpack/babel settings, nothing should change. Once it's out, you'll just need to upgrade Gatsby + any plugins you've added.

We'd love your help tackling the various TODOs. We're working on the v2 branch so jump in and start helping.

Check out the contributions docs if you're new to developing with Gatsby https://www.gatsbyjs.org/docs/how-to-contribute/

If there's not an issue already created for a TODO below, create one with any your ideas and ask for feedback/guidance.

TODOs

  • Ongoing task—keep v2 branch up-to-date with master
  • Upgrade to webpack 3 (the initial PR only got it to webpack 2)
  • meta — copy good ideas from CRA
  • add CRA's overlay
  • add CRA's eslint setup (the above two might be doable in v1 as well)
  • add CRA's local network address feature
  • "Reduxify" the API for modifying Babel configs
  • Update Gatsby's caching setup for webpack 3
  • Convert all node IDs generated by source/transformer plugins to use UUID v5 so people don't depend on them for anything
  • Convert pathContext to pageContext Change prop name "pathContext" to "pageContext" #1456
  • remove global CLI capability from gatsby package
  • all APIs should require the full object. This is especially important for deleting things as if you're responding to events, once the object is deleted, you can't access the object other than if it's passed to the event.
  • think more about bulk APIs e.g. deleteNodes. Is potential efficiency gains worth extra code complexity? Could just standardize on bulk api and remove single shot API.
  • Use UUID version 5 for node IDs in source/transformer plugins Use UUID version 5 for node IDs in source/transformer plugins #1853

Other breaking changes we should make? Add a comment!

@jquense
Copy link
Contributor

jquense commented Aug 15, 2017

maybe pass through the graphql in the plugin api to avoid some of the multiple instance issues?

@totsteps
Copy link

totsteps commented Aug 16, 2017

Serve site over the local network.
Add service worker. Not sure, but having such a functionality out of the box is great.

@KyleAMathews
Copy link
Contributor Author

Serve site over the network

@gagan0723 not sure what you mean here?

@KyleAMathews
Copy link
Contributor Author

For service workers, there's gatsby-plugin-offline https://www.gatsbyjs.org/packages/gatsby-plugin-offline/

@totsteps
Copy link

Serve site over the network

@KyleAMathews What I mean is, is there a way to serve site on the local network so that it may be viewed on other devices connected to the same network.

@KyleAMathews
Copy link
Contributor Author

Oh ok, so like Create React App, tell you the address you can use to access the site. No that is a great idea. Adding it.

@KyleAMathews
Copy link
Contributor Author

Closing this in favor of #2641

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

No branches or pull requests

3 participants