Skip to content

Fix compatibility with Symfony 4.3 #311

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 2 commits into from
Jun 24, 2019
Merged

Conversation

albingi
Copy link
Contributor

@albingi albingi commented Jun 18, 2019

The block name change from panelContent to panel

The block name change from panelContent to panel
@bocharsky-bw
Copy link
Member

Hey @albingi thank you for fixing it! Btw, do you have any reference to this change? Actually, it might be a possible BC break for those who are not on Symfony 4.3, so probably we should workaround this in some way

@albingi
Copy link
Contributor Author

albingi commented Jun 18, 2019

Yes, this is about this commit :
symfony/web-profiler-bundle@a92e0ba#diff-3e3f9f89f28a037c50c1714c5a21b3c4

@Nyholm
Copy link
Member

Nyholm commented Jun 24, 2019

@NullIsNot0 Could you review this?

Copy link
Contributor

@fsoedjede fsoedjede left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've tested and this change fix the issue

@bocharsky-bw
Copy link
Member

@Nyholm FYI I suppose only you can merge PRs in php-translation organization if tests are broken. Other core members can just merge PRs if at least one core member approved it (except the author of PR) and when tests passed. As you can see, all the tests are broken lately, so I opened #313 to fix them. Would be good to have your review on it

Copy link
Contributor

@NullIsNot0 NullIsNot0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I confirm, that changes work for Symfony 4.3

@Nyholm Nyholm merged commit ffca135 into php-translation:master Jun 24, 2019
@Nyholm
Copy link
Member

Nyholm commented Jun 24, 2019

Thank you for the reviews and for the fix

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants