Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Serializer][WebProfilerBundle] Show serializer collector info in toolbar #46566

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

Merged
merged 1 commit into from
Jun 6, 2022

Conversation

ogizanagi
Copy link
Contributor

Q A
Branch? 6.2
Bug fix? no
New feature? yes
Deprecations? no
Tickets N/A
License MIT
Doc PR N/A

Capture d’écran 2022-06-03 à 10 22 41

The numbers themselves are, most of the time, not much important, but knowing the serializer was called and which methods at a glance is helpful and allows to link directly to the Serializer panel.

@carsonbot
Copy link

Hey!

I think @mtarld has recently worked with this code. Maybe they can help review this?

Cheers!

Carsonbot

@fabpot
Copy link
Member

fabpot commented Jun 6, 2022

Thank you @ogizanagi.

@fabpot fabpot merged commit b44831b into symfony:6.2 Jun 6, 2022
@ogizanagi ogizanagi deleted the serializer-profiler-toolbar branch June 6, 2022 11:53
ogizanagi added a commit that referenced this pull request Jun 6, 2022
This PR was merged into the 6.2 branch.

Discussion
----------

[Serializer][WebProfilerBundle] Fix broken toolbar

| Q             | A
| ------------- | ---
| Branch?       | 6.2
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       | -
| License       | MIT
| Doc PR        | -

It seems that #46566 broke the toolbar:

![image](https://user-images.githubusercontent.com/2445045/172192538-04220c57-57ef-4fc3-b203-db431a319b10.png)

Commits
-------

053a5ee [Serializer][WebProfilerBundle] Fix broken toolbar
@fabpot fabpot mentioned this pull request Oct 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants