Skip to content

feat(ssr): pass additional documentHead via serverData #7770

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

Merged
merged 2 commits into from
Aug 4, 2025

Conversation

wmertens
Copy link
Member

@wmertens wmertens commented Aug 3, 2025

This will mostly be useful for the Vite environment API changes to allow injecting vite dev scripts, but seems useful in general.

@wmertens wmertens requested review from a team as code owners August 3, 2025 09:01
@wmertens wmertens force-pushed the documenthead-from-render branch from 61ff48a to 7dd2718 Compare August 3, 2025 11:09
Copy link

changeset-bot bot commented Aug 3, 2025

🦋 Changeset detected

Latest commit: bbfc9e4

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@wmertens wmertens force-pushed the documenthead-from-render branch 2 times, most recently from 15970d7 to 0b2d357 Compare August 3, 2025 13:00
Copy link
Contributor

github-actions bot commented Aug 3, 2025

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name Status Preview Last Commit
qwik-docs ✅ Ready (View Log) Visit Preview 2fd3833

@wmertens wmertens force-pushed the documenthead-from-render branch from 0b2d357 to 2fd3833 Compare August 3, 2025 17:18
 the createRenderer type was already updated in the previous commit
@wmertens wmertens force-pushed the documenthead-from-render branch from 2fd3833 to bbfc9e4 Compare August 3, 2025 20:35
@wmertens wmertens merged commit 1274d78 into build/v2 Aug 4, 2025
18 checks passed
@wmertens wmertens deleted the documenthead-from-render branch August 4, 2025 05:46
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.

2 participants