-
-
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
[WebBundle] Fix invalid html markup for payments forms #2096
Conversation
@pjedrzejewski @Arn0d ping... |
Hi, I think too that hide the form is the best solution ;) |
{{ buttons.edit(path('sylius_backend_payment_update', {'id': payment.id})) }} | ||
{%- endif %} | ||
|
||
{% for transition in ['cancel', 'complete', 'refund'] if sm_can(payment, transition, 'sylius_payment') -%} |
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.
Why not create a twig partial for this form ?
This is the same in the _details view, no, ? except for the hr tag.
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.
Yeah, maybe it makes sense?
Added easier way to change states & hide editing for non super admins
720583e
to
4ec7178
Compare
@pjedrzejewski @Arn0d ping... |
@pjedrzejewski Can we get this fix merged? |
[WebBundle] Fix invalid html markup for payments forms
Thanks Joseph! 👍 |
Added easier way to change states & hide editing for non super admins.
Fixes #1993, replaces #2005.