Skip to content

[WebProfilerBundle] Tweak Mailer panel rendering #49400

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
Feb 17, 2023

Conversation

1ed
Copy link
Contributor

@1ed 1ed commented Feb 15, 2023

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

Before

Screenshot from 2023-02-15 23-03-29

After

Screenshot from 2023-02-15 23-04-05

@1ed
Copy link
Contributor Author

1ed commented Feb 16, 2023

Should I fix the encoding stuff in 5.4 too (the tabs are different, so I think that's ok there)? It will cause conflicts, because the redesign.

@fabpot
Copy link
Member

fabpot commented Feb 17, 2023

@1ed Having a PR for 5.4 would be great.

@fabpot fabpot force-pushed the fix-profiler-email-headers-6.2 branch from 2fdabb9 to 126678b Compare February 17, 2023 11:07
@fabpot
Copy link
Member

fabpot commented Feb 17, 2023

Thank you @1ed.

@fabpot fabpot merged commit 66966ed into symfony:6.2 Feb 17, 2023
@1ed 1ed deleted the fix-profiler-email-headers-6.2 branch February 17, 2023 17:06
fabpot added a commit that referenced this pull request Feb 20, 2023
…aders (1ed)

This PR was merged into the 5.4 branch.

Discussion
----------

[WebProfilerBundle] Render original (not encoded) email headers

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

Backported version of #49400

Commits
-------

2c5abc3 [WebProfilerBundle] Render original (not encoded) email headers
@fabpot fabpot mentioned this pull request Feb 28, 2023
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.

4 participants