Skip to content

Update profiler.rst #12623

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Nov 14, 2019
Merged

Update profiler.rst #12623

merged 1 commit into from
Nov 14, 2019

Conversation

noniagriconomie
Copy link
Contributor

@noniagriconomie noniagriconomie commented Nov 12, 2019 •
edited
Loading

No need to add the header Symfony-Debug-Toolbar-Replace if the request is not an ajax one

@javiereguiluz javiereguiluz added this to the 4.3 milestone Nov 14, 2019
@OskarStark OskarStark changed the base branch from master to 4.3 November 14, 2019 14:22
@OskarStark
Copy link
Contributor

Thank you Antoine.

OskarStark added a commit that referenced this pull request Nov 14, 2019
@OskarStark
minor #12623 Update profiler.rst (noniagriconomie)
9076529 
This PR was submitted for the master branch but it was merged into the 4.3 branch instead (closes #12623).

Discussion
----------

Update profiler.rst

No need to add the header `Symfony-Debug-Toolbar-Replace` if the request is not an ajax one

Commits
-------

27f7029 Update profiler.rst
@OskarStark OskarStark merged commit 27f7029 into symfony:4.3 Nov 14, 2019
@noniagriconomie noniagriconomie deleted the patch-3 branch November 14, 2019 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@javiereguiluz javiereguiluz javiereguiluz approved these changes

Assignees
No one assigned
Labels
Profiler Status: Reviewed
Projects
None yet
Milestone
4.3
Development

Successfully merging this pull request may close these issues.
4 participants
@noniagriconomie @OskarStark @javiereguiluz @carsonbot