Skip to content

Fix Crawlinks issue during build - #459

Merged
LadyBluenotes merged 3 commits into
solidjs:mainfrom
atilafassina:remove-prerender
Feb 12, 2024
Merged

LadyBluenotes merged 3 commits into
solidjs:mainfrom
atilafassina:remove-prerender

Conversation

@atilafassina

@atilafassina atilafassina commented Feb 12, 2024 •

Copy link
Copy Markdown
Member

Nitro can't crawl <A> links and it's causing bad prerendering. It may be causing a few cascade issues with the deployed site.

possibly related to: #455

@netlify

netlify Bot commented Feb 12, 2024 •

Copy link
Copy Markdown

✅ Deploy Preview for solid-docs ready!

Name Link
🔨 Latest commit 2252af4
🔍 Latest deploy log https://app.netlify.com/sites/solid-docs/deploys/65ca3fa0213e9c00086cef00
😎 Deploy Preview https://deploy-preview-459--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 added the bug Something isn't working label Feb 12, 2024
@LadyBluenotes
LadyBluenotes merged commit 0f4cec3 into solidjs:main Feb 12, 2024
@atilafassina
atilafassina deleted the remove-prerender branch February 12, 2024 21:33
ryansolid added a commit that referenced this pull request Oct 22, 2026
…1665)

* docs: sync diagnostics pages and the Vite plugin's test environment

- Regenerate the diagnostics reference from solid next 6cca8b57: adds
  STORE_WRITE_OUTSIDE_SETTER (#4194) and UNCOMMITTED_INPUT_OVERWRITTEN
  (#4205), drops BINDING_SLOT_POSITION (binding slots were removed), and
  picks up skill edits to EFFECT_RELAY_TEAR, LOWERCASE_EVENT_ATTRIBUTE,
  REACTIVITY_HALTED, SSR_BOUNDARY_WATERFALL and SSR_LIVE_HOLE_LATCHED.
  The generator's FIX_STARTS entry for SSR_BOUNDARY_WATERFALL follows the
  skill's new capitalization, and the dead BINDING_SLOT_POSITION entry goes.
- The Vite plugin no longer defaults test.environment to jsdom
  (vite-plugin-solid #459): the testing guide drops the "browser mode still
  needs jsdom" note and troubleshooting entry, browser mode no longer
  installs jsdom, and a new entry covers `document is not defined`. The
  plugin options reference and the Solid 1 migration guide say to set the
  environment.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01R9d4iekBWmtH343gNAGJpd

* docs: regenerate diagnostics against solid-js@2.0.0-rc.17

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01R9d4iekBWmtH343gNAGJpd

---------

Co-authored-by: Claude <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants