Skip to content

Conversation

@MichaelDeBoey
Copy link
Contributor

@MichaelDeBoey MichaelDeBoey commented Apr 13, 2021

We already use gatsby-remark-embedder in some places, but we're not using it fully (yet).

@MichaelDeBoey MichaelDeBoey changed the title chore: use gatsby-remark-embedder to transform all embeds in docs chore: use gatsby-remark-embedder fully to transform all embeds in docs Apr 13, 2021
@codesandbox-ci
Copy link

codesandbox-ci bot commented Apr 13, 2021

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 92a2f8b:

Sandbox Source
Notifications Test Configuration

@lbogdan lbogdan temporarily deployed to pr5658 April 13, 2021 11:12 Inactive
@lbogdan
Copy link
Contributor

lbogdan commented Apr 13, 2021

Build for latest commit 92a2f8b is at https://pr5658.build.csb.dev/s/new.

Copy link
Contributor

@danilowoz danilowoz left a comment

Choose a reason for hiding this comment

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

Hey @MichaelDeBoey, thanks for your contribution! However, it doesn't really work (see my comments). Would you like to care of this, or can we close this?

@MichaelDeBoey MichaelDeBoey requested a review from danilowoz May 12, 2021 13:49
@MichaelDeBoey MichaelDeBoey force-pushed the use-gatsby-remark-embedder-fully branch from 653ef4b to e5a3f28 Compare May 13, 2021 10:58
@MichaelDeBoey
Copy link
Contributor Author

@danilowoz All tests are green

Copy link
Contributor

@danilowoz danilowoz left a comment

Choose a reason for hiding this comment

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

All good!

@danilowoz danilowoz merged commit bb81765 into codesandbox:master May 13, 2021
@MichaelDeBoey MichaelDeBoey deleted the use-gatsby-remark-embedder-fully branch May 13, 2021 14:29
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

Successfully merging this pull request may close these issues.

3 participants