Skip to content

fix callouts#1157

Merged
LadyBluenotes merged 1 commit intosolidjs:mainfrom
Brendonovich:fix-callouts
Apr 9, 2025
Merged

fix callouts#1157
LadyBluenotes merged 1 commit intosolidjs:mainfrom
Brendonovich:fix-callouts

Conversation

@Brendonovich
Copy link
Copy Markdown
Contributor

  • I have read the Contribution guide
  • This PR references an issue (except for typos, broken links, or other minor problems)

Description(required)

Hydration errors caused by props.children being evaluated twice in DirectiveContainer are being surfaced as 404s. This PR just ensures that props.children is only evaluated one, using the result of the children() for all cases.

Related issues & labels

@bolt-new-by-stackblitz
Copy link
Copy Markdown

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@netlify
Copy link
Copy Markdown

netlify bot commented Apr 9, 2025

Deploy Preview for solid-docs ready!

Name Link
🔨 Latest commit ffa29ce
🔍 Latest deploy log https://app.netlify.com/sites/solid-docs/deploys/67f5e2c2af9fbb0008ecdc21
😎 Deploy Preview https://deploy-preview-1157--solid-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@LadyBluenotes LadyBluenotes merged commit fd89e2d into solidjs:main Apr 9, 2025
7 checks passed
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.

[Bug]: "Config Base Routing" page hits a soft-404

2 participants