Skip to content

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