Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

InternalError: too much recursion for v1.21+ #614

Closed
czernika opened this issue Mar 11, 2024 · 6 comments · Fixed by #615
Closed

InternalError: too much recursion for v1.21+ #614

czernika opened this issue Mar 11, 2024 · 6 comments · Fixed by #615

Comments

@czernika
Copy link

I'm using Laravel with Orchid admin panel and Laravel debugbar

Since last update I've noticed that debugbar dissapears when I navigate to some page - it works on initial load however. At first I thought it something with project code/deps, but digging deeper I found out it was this particular package (which is laravel-debugbar dependency) and more specifically version 1.21+ (I've installed version 1.20.1 and debugbar start working again)

Orchid is using Hotwire under the hood so I believe it has to be something with it

Снимок экрана от 2024-03-11 18-45-03

@erikn69
Copy link
Contributor

erikn69 commented Mar 11, 2024

Hi, please try #613 (comment)
CTRL + SHIFT + R for reload js cache

@czernika
Copy link
Author

@erikn69 Ah yes thanks. Problem was specific for 1.21.0 (I didn't noticed 1.21.1 release about hour ago as start debugging earlier - with 1.21.1 everything back to normal)

@erikn69
Copy link
Contributor

erikn69 commented Mar 11, 2024

1.21.1 was a regression,
I was referring to the fix that I proposed on #613 (comment)

@erikn69
Copy link
Contributor

erikn69 commented Mar 11, 2024

I added #615,
If you could confirm that it works correctly for you it would be very helpful,
so if it does merge you won't have the same problem again.

@czernika
Copy link
Author

@erikn69 on 1.21.0 with proposed changes from #615 it works

@erikn69
Copy link
Contributor

erikn69 commented Mar 11, 2024

thank you very much ❤️

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 a pull request may close this issue.

2 participants