-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Multiple integration bugfixes #5865
Conversation
nihco2
commented
Jul 12, 2016
•
edited
Loading
edited
Questions | Answers |
---|---|
Branch? | develop |
Description? | multiple bugfixes |
Type? | bug fix |
Category? | CO |
BC breaks? | no |
Deprecations? | no |
Fixed ticket? | filtered bug list here: http://bit.ly/29sWf3I |
How to test? | Check the bug list link above |
7255df6
to
62dbd32
Compare
@@ -124,7 +124,7 @@ public function buildForm(FormBuilderInterface $builder, array $options) | |||
'options' => array(), | |||
'locales' => $this->locales, | |||
'hideTabs' => true, | |||
'label' => $this->translator->trans('Label when out of stock', array(), 'Admin.Catalog.Feature'), | |||
'label' => $this->translator->trans('Displayed text when out of stock', array(), 'Admin.Catalog.Feature'), |
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.
ping @AlexEven (just in case)
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.
Thanks! I updated it in another PR #5901
Status : 'QA approved' Cool thanks :) All the tickets have been tested and closed |
Status : 'PM approved' |