-
-
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
Implement Shipping Method Index/Create/Update pages #16129
Implement Shipping Method Index/Create/Update pages #16129
Conversation
Bunnyshell Preview Environment deletedAvailable commands:
|
5f1885f
to
2f734f5
Compare
d9d0767
to
b617846
Compare
/bns:start |
src/Sylius/Bundle/AdminBundle/Resources/views/Shared/form_theme.html.twig
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/AdminBundle/Resources/views/Shared/form_theme.html.twig
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/AdminBundle/Resources/views/Shared/form_theme.html.twig
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/ShippingBundle/Form/Type/ShippingMethodType.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/AdminBundle/TwigComponent/ShippingMethod/FormComponent.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/AdminBundle/Resources/views/Shared/form_theme.html.twig
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/AdminBundle/Resources/views/ShippingMethod/form.html.twig
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/ShippingBundle/Form/Type/ShippingMethodRuleCollectionType.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/ShippingBundle/Resources/config/services/form.xml
Outdated
Show resolved
Hide resolved
9d8dad7
to
38a939d
Compare
src/Sylius/Behat/Context/Api/Admin/ManagingShippingMethodsContext.php
Outdated
Show resolved
Hide resolved
src/Sylius/Behat/Context/Ui/Admin/ManagingShippingMethodsContext.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/AdminBundle/Resources/config/services/form.xml
Outdated
Show resolved
Hide resolved
…rs.feature scenarios
… in the form element
5098eb6
to
3f77aef
Compare
public function __construct( | ||
private IndexPageInterface $indexPage, | ||
private CreatePageInterface $createPage, | ||
private UpdatePageInterface $updatePage, | ||
private CurrentPageResolverInterface $currentPageResolver, | ||
private NotificationCheckerInterface $notificationChecker, | ||
private FormElementInterface $shippingMethodFormElement, |
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.
private FormElementInterface $shippingMethodFormElement, | |
private FormElementInterface $formElement, |
🤔
/** | ||
* @Then I should be notified that name is required | ||
*/ | ||
public function iShouldBeNotifiedThatNameIsRequired($localeCode = 'en_US'): void |
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.
public function iShouldBeNotifiedThatNameIsRequired($localeCode = 'en_US'): void | |
public function iShouldBeNotifiedThatNameIsRequired(string $localeCode = 'en_US'): void |
} | ||
|
||
/** | ||
* @Then I should be notified that name is required |
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.
Then
methods should be after When
ones
use Sylius\Behat\Service\DriverHelper; | ||
use Sylius\Behat\Service\TabsHelper; | ||
use Sylius\Component\Core\Formatter\StringInflector; | ||
use Webmozart\Assert\Assert; | ||
|
||
class CreatePage extends BaseCreatePage implements CreatePageInterface |
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.
Both CreatePage
and UpdatePage
for shipping method could be probably removed as they are empty now
<tag name="form.type_extension" /> | ||
</service> | ||
|
||
<service id="sylius.form.extension.shipping_method_rule" class="Sylius\Bundle\AdminBundle\Form\Extension\ShippingMethodRuleTypeExtension"> |
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.
<service id="sylius.form.extension.shipping_method_rule" class="Sylius\Bundle\AdminBundle\Form\Extension\ShippingMethodRuleTypeExtension"> | |
<service id="sylius_admin.form.extension.shipping_method_rule" class="Sylius\Bundle\AdminBundle\Form\Extension\ShippingMethodRuleTypeExtension"> |
bootstrap-admin-panel