search

ndungtse / next13-progressbar

A simple Next.js progressbar component using NProgress with support of next.js >=13 app dir.
https://next13-progressbar.vercel.app
MIT License
91 stars 3 forks source link

Missing Suspense boundary with useSearchParams #24

Closed HighError closed 8 months ago

HighError commented 8 months ago

Describe the bug Missing Suspense boundary with useSearchParams

To Reproduce Steps to reproduce the behavior:

  1. Add Next13ProgressBar to app
  2. Build

Expected behavior Build ok.

Screenshots image

Additional context You need to wrap the component in Suspend. I think it's worth adding this to the library so that the developer doesn't have to think about it. image

ndungtse commented 8 months ago

This issue is the same as #18 which I moved into discussion if the feature can be added but it looks like it might be helpful to be included in package. I'll try to include it in the next versions. Thanks in advance @HighError .

MrLoh commented 8 months ago

Just wasted 4 hours tracking this down.