-
Notifications
You must be signed in to change notification settings - Fork 15
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
IBX-1742: Dropdown not working in Shipping Management - dynamic dropdowns #122
IBX-1742: Dropdown not working in Shipping Management - dynamic dropdowns #122
Conversation
tischsoic
commented
Dec 20, 2021
Question | Answer |
---|---|
Tickets | https://issues.ibexa.co/browse/IBX-1742 |
Bug fix? | yes/no |
New feature? | yes/no |
BC breaks? | yes/no |
Tests pass? | yes |
Doc needed? | yes/no |
License | GPL-2.0 |
@@ -68,7 +71,7 @@ | |||
|
|||
{% include '@ibexadesign/ui/component/dropdown/dropdown_selected_item.html.twig' with { | |||
value: first_choice.value, | |||
label: _self.get_translated_label(first_choice.label), | |||
label: _self.get_translated_label(first_choice.label, false), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was no translation_domain
so I guess we want to pass false
here (as the condition in macro is translation_domain is same as(false)
, which is false for not defined translation_domain
)?
Can you confirm @GrabowskiM? :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We want to use translation_domain
variable, as in all other uses of this macro (for example line 67)
@@ -68,7 +71,7 @@ | |||
|
|||
{% include '@ibexadesign/ui/component/dropdown/dropdown_selected_item.html.twig' with { | |||
value: first_choice.value, | |||
label: _self.get_translated_label(first_choice.label), | |||
label: _self.get_translated_label(first_choice.label, false), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We want to use translation_domain
variable, as in all other uses of this macro (for example line 67)
a760317
to
1797527
Compare