-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[Addressing] Province rendering tweaks #3953
[Addressing] Province rendering tweaks #3953
Conversation
NoResponseMate
commented
Jan 22, 2016
Q | A |
---|---|
Bug fix? | no |
New feature? | yes |
BC breaks? | yes |
Deprecations? | no |
Fixed tickets | #3902 |
License | MIT |
Doc PR | - |
@@ -30,6 +30,9 @@ public function buildForm(FormBuilderInterface $builder, array $options) | |||
->add('name', 'text', array( | |||
'label' => 'sylius.form.province.name', | |||
)) | |||
->add('abbreviation', 'text', array( | |||
'label' => 'sylius.form.province.abbreviation', |
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.
It's not required so we should set it to false here.
Looks good to me, let's fix the build, merge it and add caching in another PR. |
->willReturn($provinceForm); | ||
|
||
|
||
|
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.
Redundant blank spaces
Add fallback to |
Thanks for getting this done so quickly :) 👍 |
180fb7d
to
1159f2c
Compare
[Addressing] Province rendering tweaks
Thank you Jasiek! 👍 :) |
…lore [Addressing] Province rendering tweaks
…lore [Addressing] Province rendering tweaks