feat(link)!: require URL resolver injection - #751
Conversation
🦋 Changeset detectedLatest commit: 5b41f8b The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
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 |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (7)
Disabled knowledge base sources:
📝 WalkthroughSummary by CodeRabbit
WalkthroughThe Link package now uses a public ChangesLink URL resolver API
Estimated code review effort: 3 (Moderate) | ~20 minutes Possibly related PRs
Suggested reviewers: ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
stackflow-docs | 5b41f8b | Commit Preview URL | Aug 10 2026, 05:41 AM |
Deploying stackflow-demo with
|
| Latest commit: |
c622af5
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://e384d496.stackflow-demo.pages.dev |
| Branch Preview URL: | https://feature-fep-2677-link-resolv.stackflow-demo.pages.dev |
commit: |
Summary
Linkconsume the resolver supplied throughLinkUrlResolverProvider.@stackflow/plugin-history-sync, allowing history-sync or another routing implementation to provide the resolver.Resolves FEP-2677.