Gatsby 3 touchNode deprecation warning

[njosefbeck]

Gatsby 3 touchNode deprecation warning

[njosefbeck]

This should be resolved in the latest version, so closing this now. Feel free to reopen if you continue to experience the issue.

[njosefbeck]

@brxck So I'm a little fuzzy on the implementation here, but in this file would I then just pass the node into saveNodeFromGarbageCollection, or maybe touchNode orrr does cachedFileData have a reference to the node? I can dig in at some point, but thought I'd ask here first in case you have more insight.

gatsby-source-stripe/src/helpers/checkCache.js

Line 12 in 9166458

saveNodeFromGarbageCollection(touchNode, cachedFileData.fileNodeId);

[brxck]

I'm also fuzzy here. My naive impression is that there are two different nodes within checkCache(). node is the Stripe resource node, while the node referred to bycachedFileData.fileNodeId is the File node that would belong to it.

I don't think we actually have a reference to the File node being touched, so you might have to retrieve it via the getNode helper? This code and these Gatsby APIs are new to me though, so take that with a grain of salt 😅

[njosefbeck]

Ah yeah maybe it's as simple as passing:

gatsby-source-stripe/src/helpers/checkCache.js

Line 9 in 9166458

const fileNode = getNode(cachedFileData.fileNodeId);

this node into saveNodeFromGarbageCollection and thus into touchNode. I'll try that and see how it goes.