Max Prilutskiy
Max Prilutskiy
@justinwaite @kentcdodds @yracnet folks, any chance we can solve this any time soon? Any support needed with anything? @justinwaite have you put together a PR you talked about or not...
A year after, the issue is still there. It's seems to be possible to invalidate store after mutation, but it isn't when using `commitLocalUpdate`
Sort of related: https://github.com/pbteja1998/remix-auth-email-link/pull/47
haha, good point @tanzyy96 🤣 would it be a good option to create a temp channel first, just for testing?
I can confirm that this issue completely breaks a Remix js web app. We upgraded to the latest nextui, and our whole platform stopped functioning because of broken links. Please...
@johnmangan yep, exactly this! Here's a stackblitz repro: https://stackblitz.com/edit/remix-run-remix-nrotvl?file=app%2Froot.tsx cc @wingkwong
gotcha, @wingkwong , thank you very much! any rough estimate when the new version will be rolled out? could help with planning on our side.
@wingkwong i've just added these lines to package.json ``` "resolutions": { "@nextui-org/link": "0.0.0-canary-20240520130845", "@nextui-org/system": "0.0.0-canary-20240520130845" } ``` and the error is still there, I'm afraid! https://stackblitz.com/edit/remix-run-remix-nrotvl?file=package.json
@wingkwong thanks for sharing the example! Links now indeed trigger navigation, but I'm afraid they now cause the page to reload entirely. Can you confirm you see the same?
Amazing, thank you very much @wingkwong! Everything now works! 🎉