fix(solid-router): restore MatchRoute reactivity#6911
fix(solid-router): restore MatchRoute reactivity#6911Sheraff wants to merge 1 commit intorefactor-signalsfrom
Conversation
Track MatchRoute options and child rendering inside Solid memos so navigation updates and reactive params cannot get stuck on the initial match result.
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Comment Tip CodeRabbit can use TruffleHog to scan for secrets in your code with verification capabilities.Add a TruffleHog config file (e.g. trufflehog-config.yml, trufflehog.yml) to your project to customize detectors and scanning behavior. The tool runs only when a config file is present. |
|
View your CI Pipeline Execution ↗ for commit c957c51
☁️ Nx Cloud last updated this comment at |
Bundle Size Benchmarks
Trend sparkline is historical gzip bytes ending with this PR measurement; lower is better. |
Summary
MatchRouteso route changes do not leave render-prop or plain children stuck on the initial match resultuseMatchRouteoption destructuring into the memoized scope so reactivetoandparamsinputs are tracked correctlyMatchRoutechild formsTesting
CI=1 NX_DAEMON=false pnpm nx run @tanstack/solid-router:test:unit --outputStyle=stream --skipRemoteCacheCI=1 NX_DAEMON=false pnpm nx run @tanstack/solid-router:test:types --outputStyle=stream --skipRemoteCache