Skip to content

Conversation

@ascorbic
Copy link
Contributor

Remote images are no more experimental than the rest of the image plugin now, so I think it's fine to unflag them.

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Jan 14, 2021
if (isRemoteURL(src)) {
let createRemoteFileNode
try {
;({ createRemoteFileNode } = require(`gatsby-source-filesystem`))
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, I know, @laurieontech. This is just easier to read.

Copy link
Contributor

Choose a reason for hiding this comment

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

Haha I agree, don’t worry

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I mean, I'm grateful that I now know how to do it!

@ascorbic ascorbic added bot: merge on green Gatsbot will merge these PRs automatically when all tests passes topic: media Related to gatsby-plugin-image, or general image/media processing topics and removed status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer labels Jan 14, 2021
@gatsbybot gatsbybot merged commit e72533d into master Jan 14, 2021
@gatsbybot gatsbybot deleted the chore/unflag-remote-images branch January 14, 2021 14:52
This was referenced Jan 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bot: merge on green Gatsbot will merge these PRs automatically when all tests passes topic: media Related to gatsby-plugin-image, or general image/media processing topics

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants