Skip to content

Fix for nested routes rendering issue#93

Open
kevin-coscarelli wants to merge 2 commits intovendurehq:masterfrom
kevin-coscarelli:fix-route-generation-issue
Open

Fix for nested routes rendering issue#93
kevin-coscarelli wants to merge 2 commits intovendurehq:masterfrom
kevin-coscarelli:fix-route-generation-issue

Conversation

@kevin-coscarelli
Copy link

Fix for nested routes rendering issue as described here: #92

Removing manual route generation seems to have fiuxed the issue:
image

@netlify
Copy link

netlify bot commented Dec 24, 2024

Deploy Preview for inspiring-boba-355f4d ready!

Name Link
🔨 Latest commit 6bca21a
🔍 Latest deploy log https://app.netlify.com/sites/inspiring-boba-355f4d/deploys/676aa7e075a70500088f4d6d
😎 Deploy Preview https://deploy-preview-93--inspiring-boba-355f4d.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.

@netlify
Copy link

netlify bot commented Dec 24, 2024

Deploy Preview for remix-vendure ready!

Name Link
🔨 Latest commit 6bca21a
🔍 Latest deploy log https://app.netlify.com/sites/remix-vendure/deploys/676aa7e01be42c0008255b22
😎 Deploy Preview https://deploy-preview-93--remix-vendure.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.

@dylviz
Copy link
Contributor

dylviz commented Feb 5, 2025

This looks good to me

axbegue added a commit to axbegue/storefront-remix-starter that referenced this pull request Feb 12, 2025
@wayinone
Copy link

wayinone commented Feb 19, 2025

Much much appreciated the effort.

@kohane27
Copy link

@dylviz Please merge this 🙏

@dylviz
Copy link
Contributor

dylviz commented Apr 13, 2025

@dylviz Please merge this 🙏

I think we'd need Michael for that. He's pretty busy though, might be faster to just throw the fix into your code if it's a blocker.

@nikbrunner
Copy link

I had the same issue. I suspect this is a regression from #90.

wayinone added a commit to wayinone/ddiligence-front that referenced this pull request Jun 2, 2025
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.

5 participants