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

feat(gatsby-link,gatsby-plugin-image,gatsby-script): Client export directive #36470

Merged
merged 2 commits into from Aug 26, 2022

Conversation

tyhopp
Copy link
Contributor

@tyhopp tyhopp commented Aug 26, 2022

Description

Mark client components with the "client export" directive for usage in RSC.

microbundle doesn't allow us to hard code the string in the files, it strips out all comments by default and doesn't have a way to allow only certain comments.

As a quick workaround I wrote a module to do this for us in a post build script - https://github.com/tyhopp/prepend-directive/blob/main/index.js.

Documentation

N/A

Related Issues

[sc-54927]

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Aug 26, 2022
@tyhopp tyhopp removed the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Aug 26, 2022
@tyhopp tyhopp requested review from wardpeet and removed request for wardpeet August 26, 2022 07:57
@tyhopp tyhopp marked this pull request as draft August 26, 2022 08:03
@tyhopp tyhopp marked this pull request as ready for review August 26, 2022 10:00
@wardpeet wardpeet merged commit 320efb7 into master Aug 26, 2022
@wardpeet wardpeet deleted the feat/client-export-directive branch August 26, 2022 13:18
@tyhopp tyhopp added this to the Gatsby 5 milestone Sep 21, 2022
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.

None yet

2 participants