framework
framework copied to clipboard
docs: Update middleware example redirect syntax
...because navigateTo didn't work for me when deploying as a static site, but returning the target path string directly did work
🔗 Linked issue
❓ Type of change
- [x] 📖 Documentation (updates to the documentation or readme)
- [ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
- [ ] 👌 Enhancement (improving an existing functionality like performance)
- [ ] ✨ New feature (a non-breaking change that adds functionality)
- [ ] ⚠️ Breaking change (fix or feature that would cause existing functionality to change)
📚 Description
📝 Checklist
- [ ] I have linked an issue or discussion.
- [ ] I have updated the documentation accordingly.
Deploy Preview for nuxt3-docs ready!
Name | Link |
---|---|
Latest commit | 4684ab99511ecc288ac5711a514ae4d9ded23869 |
Latest deploy log | https://app.netlify.com/sites/nuxt3-docs/deploys/62e1c60e75537e0008082a9a |
Deploy Preview | https://deploy-preview-6193--nuxt3-docs.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site settings.
You're right, returning the string directly will work, and is a good workaround. But our intent is that navigateTo
should work also, and be more semantic. Would you raise a bug report with a reproduction so we can address the underlying issue? 🙏
I'm struggling to create a minimal workaround of this 😩 but the bug is still hitting my primary app so I'm working on a "top-down" approach removing dependencies until the bugs stops. Meanwhile rc8 was released which I thought might include an indirect fix but I can't test yet because of nuxt/nuxt.js#14532 . I'll update when I can
@jonnyparris Do you mind to make a an issue with reproduction to track? 🙏🏼 Indeed this has to be solved to allow navigateTo
usage.
Just fyi - I indirectly fixed my particular navigation issue with thanks to workarounds in this issue thread https://github.com/nuxt-modules/supabase/issues/28