Skip to content

Ajax requests are not considered #41

Description

@fduarte42

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?

Activity

  1. tyrsson commented on Aug 15, 2023

    @tyrsson

    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.

  2. fduarte42 commented on Aug 16, 2023

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

  3. tyrsson commented on Aug 18, 2023

    @tyrsson

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

  4. fduarte42 commented on Aug 18, 2023

    @fduarte42
    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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions