Closed mohitb35 closed 2 weeks ago
The latest updates on your projects. Learn more about Vercel for Git ↗︎
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
planet-webapp-multi-tenancy-setup | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Nov 6, 2024 10:54am |
[!IMPORTANT]
Review skipped
Auto reviews are disabled on base/target branches other than the default branch.
Please check the settings in the CodeRabbit UI or the
.coderabbit.yaml
file in this repository. To trigger a single review, invoke the@coderabbitai review
command.You can disable this status message by setting the
reviews.review_status
tofalse
in the CodeRabbit configuration file.
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Changes
Technical Details
Introduced proper parameter management in
buildProjectDetailsQuery
(renamed fromupdateUrlWithParams
):Record<string, string>
for URL query paramsEnhanced
updateProjectDetailsPath
(renamed frompushWithShallow
):Unified handling of both site and plant location navigation:
Enhanced
generateProjectLink
logic (supplementary change)backNavigationUrl
to strip locale from pathRefactored
handleBackButton
logic (supplementary change)Understanding the Main Fix
The core issue stemmed from the interaction between shallow routing and our middleware:
Normal Navigation:
/de/prd/yucatan
to/sites/planet/de/prd/yucatan
locale
,slug
,p
) from the rewritten pathrouter.query
but not in the URL query stringShallow Routing:
The fix ensures that during shallow routing:
locale
,slug
,p
) are preserved inrouter.query