framework icon indicating copy to clipboard operation
framework copied to clipboard

docs: Update middleware example redirect syntax

Open jonnyparris opened this issue 2 years ago • 2 comments

...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.

jonnyparris avatar Jul 27 '22 23:07 jonnyparris

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...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

netlify[bot] avatar Jul 27 '22 23:07 netlify[bot]

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? 🙏

danielroe avatar Jul 28 '22 10:07 danielroe

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 avatar Aug 14 '22 07:08 jonnyparris

@jonnyparris Do you mind to make a an issue with reproduction to track? 🙏🏼 Indeed this has to be solved to allow navigateTo usage.

pi0 avatar Aug 15 '22 14:08 pi0

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

jonnyparris avatar Aug 22 '22 23:08 jonnyparris