-
-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Extract HTML from view_operations.php to a Twig template #14836
Extract HTML from view_operations.php to a Twig template #14836
Conversation
Signed-off-by: Thiago Colares <thicolares@gmail.com>
…late Signed-off-by: Thiago Colares <thicolares@gmail.com>
The Travis CI build failed. I'm going to check this out in a few minutes |
Signed-off-by: Thiago Colares <thicolares@gmail.com> Signed-off-by: Thiago Colares <thicolares@gmail.com>
1808108
to
742f87e
Compare
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.
LGTM
Thank you @thicolares
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.
Hi, @thicolares. Thanks for your contribution.
Could you rename the template to templates/table/operations/view.twig
?
Signed-off-by: Thiago Colares <thicolares@gmail.com>
Signed-off-by: Thiago Colares <thicolares@gmail.com>
Signed-off-by: Thiago Colares <thicolares@gmail.com>
Signed-off-by: Thiago Colares <thicolares@gmail.com>
Signed-off-by: Thiago Colares <thicolares@gmail.com>
…thicolares/phpmyadmin into extract-html-in-view-operations-to-twig Signed-off-by: Thiago Colares <thicolares@gmail.com>
213e826
to
dbd674b
Compare
Codecov Report
@@ Coverage Diff @@
## master #14836 +/- ##
============================================
+ Coverage 50.87% 50.89% +0.01%
Complexity 14033 14033
============================================
Files 473 473
Lines 65898 65877 -21
============================================
Hits 33525 33525
+ Misses 32373 32352 -21 |
1 similar comment
Codecov Report
@@ Coverage Diff @@
## master #14836 +/- ##
============================================
+ Coverage 50.87% 50.89% +0.01%
Complexity 14033 14033
============================================
Files 473 473
Lines 65898 65877 -21
============================================
Hits 33525 33525
+ Misses 32373 32352 -21 |
Merged, thanks for your contribution! |
Description
Related to #14801
Before submitting pull request, please review the following checklist:
Signed-off-by
line as described in our DCO. This ensures that the work you're submitting is your own creation.