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

Ajax requests are not considered #41

Open
fduarte42 opened this issue Aug 14, 2023 · 4 comments
Open

Ajax requests are not considered #41

fduarte42 opened this issue Aug 14, 2023 · 4 comments

Comments

@fduarte42
Copy link

fduarte42 commented Aug 14, 2023

When using ajax requests these requests are not integrated into debugbar, because they are not considered.

This Pullrequest fixes this ( PR 40 ).

Do you have a timeframe when this will be merged and released?

@tyrsson
Copy link

tyrsson commented Aug 15, 2023

Its rendered in my application, the bars are stacked one on top of the other so you end up with one for the original request, then it stacks one for each ajax request or at least that is the behavior I was getting.

@fduarte42
Copy link
Author

Yes, correct. But this is faulty and this PR fixes that. As the debugbar module can handle ajax requests, but this bridge does not use that funcionality. The PR adds that needed parameters.

@tyrsson
Copy link

tyrsson commented Aug 18, 2023

[offtopic]
I thought it was ironic we used the same exact variable name: isAjax.
[/offtopic]

@fduarte42
Copy link
Author

The PR is not from me, but yes I did notice that. ;-)

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

No branches or pull requests

2 participants