Skip to content
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

Use translation domain of admin as default for form_help #7021

Merged
merged 1 commit into from
Apr 16, 2021

Conversation

franmomu
Copy link
Member

@franmomu franmomu commented Apr 7, 2021

Subject

I'll add tests after #7020

I am targeting this branch, because these changes are BC.

Closes #7014.

Changelog

### Fixed
- Fixed using the admin translation domain as default for `form_help` twig block.

I think we can use the same approach for form_label and do not override the entire block, just setting variables, I'll try to have a look.

@franmomu franmomu marked this pull request as draft April 7, 2021 16:51
VincentLanglet
VincentLanglet previously approved these changes Apr 14, 2021
core23
core23 previously approved these changes Apr 14, 2021
@franmomu franmomu dismissed stale reviews from core23 and VincentLanglet via f41c2b1 April 16, 2021 07:40
@franmomu franmomu force-pushed the translation_domain_form branch from c078bed to f41c2b1 Compare April 16, 2021 07:40
@franmomu franmomu marked this pull request as ready for review April 16, 2021 07:41
@VincentLanglet VincentLanglet requested a review from a team April 16, 2021 07:52
@jordisala1991 jordisala1991 merged commit 3ec8262 into sonata-project:3.x Apr 16, 2021
@jordisala1991
Copy link
Member

Thank you @franmomu

@franmomu franmomu deleted the translation_domain_form branch April 16, 2021 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Form "help" is not translated using the (default) domain of the admin.
4 participants