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.

Sign up for GitHub

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] Migration guide updates #5661

Merged
merged 27 commits into from
Jun 5, 2018
Merged

[v2] Migration guide updates #5661

Changes from 1 commit
Commits
Show all changes
27 commits
Select commit Hold shift + click to select a range
ddd19d6
Remove yarn examples for consistency with other docs
m-allanson Jun 1, 2018
8718398
Add rename responsive image queries section
m-allanson Jun 1, 2018
bf0a458
Add postcss plugins section
m-allanson Jun 1, 2018
231184a
Remove checklist (moved to issue 3986)
m-allanson Jun 1, 2018
00d68ee
Add mixed CommonJS/ES6 section
m-allanson Jun 1, 2018
8053835
Link to Gatsby webpack docs
m-allanson Jun 1, 2018
bb64597
Add node id section
m-allanson Jun 1, 2018
7afa9d9
Add section: remove explicit polyfills
m-allanson Jun 1, 2018
f616086
Add section: Change modifyBabelrc to onCreateBabelConfig
m-allanson Jun 1, 2018
163d7ea
Add section: Change `modifyWebpackConfig` to `onCreateWebpackConfig`
m-allanson Jun 1, 2018
e54d5f9
Add section: CSS is inlined automatically
m-allanson Jun 1, 2018
5413b73
Add introduction and small tweaks
m-allanson Jun 1, 2018
d10adad
Correct line numbers
m-allanson Jun 1, 2018
ca6d375
Improve explanation around movement of libs to peerDependencies
KyleAMathews Jun 1, 2018
433833a
chore: minor capitalization tweak
jlengstorf Jun 1, 2018
05b1e85
Add edit link
m-allanson Jun 4, 2018
d0cebc7
Split peer dep updates into two sections
m-allanson Jun 4, 2018
9124ad1
Declarative headings
m-allanson Jun 4, 2018
c5e1c15
Add table of contents
m-allanson Jun 4, 2018
2be2dca
Use neater Fragment syntax
m-allanson Jun 4, 2018
b8c4860
More detail on deprecation note
m-allanson Jun 4, 2018
405073e
Inline comments on code examples
m-allanson Jun 4, 2018
bfc9301
Add Jason's node ID query example
m-allanson Jun 4, 2018
6af965a
merge origin/v2-migration-guide
m-allanson Jun 4, 2018
d5beb1b
Use markdown diff syntax for before / after examples
m-allanson Jun 4, 2018
ad47463
Add 'in-progress' note about babel polyfills
m-allanson Jun 4, 2018
ed23990
Add instructions on how to install Gatsby v2
m-allanson Jun 4, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add node id section
  • Loading branch information
@m-allanson
m-allanson committed Jun 1, 2018
commit bb64597ea9251143657678472f0d7225de9c3282
9 changes: 9 additions & 0 deletions docs/docs/migrating-from-v1-to-v2.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -277,3 +277,12 @@ Mixing `import` and `module.exports` is not ok:
import foo from "foo"
module.exports = foo;
```

## Don't query nodes by ID
Copy link
Contributor

Choose a reason for hiding this comment

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

Perhaps eg "Rewrite node ID queries"?

Copy link
Contributor

Choose a reason for hiding this comment

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

Hmmm I think @m-allanson original wording is correct. There aren't "node ID queries" just queries that might try to query nodes by ID.


Source and transformer plugins now use UUIDs for IDs. If you used glob or regex to query nodes by id then you'll need to query something else.

[See the Pull Request that implemented this change](https://github.com/gatsbyjs/gatsby/pull/3807/files)

> TODO: add example
Copy link
Contributor

Choose a reason for hiding this comment

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

This is a real example from one of my projects. 😄

  query MyImageQuery {
    allImageSharp(filter: {
-     id: {regex: "/default.jpg/"}
+     sizes: {originalName: {regex: "/default.jpg/"}}
    }) {
      edges {
        node {
          id
          sizes(maxWidth: 660) {
            src
          }
        }
      }
    }
  }

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks :)