Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
started work
Comment | File | Size | Author |
---|---|---|---|
#6 | drupal_8_theme_and_twig_sprint-rethemed-1778874-6.patch | 710 bytes | crasherzvir |
#4 | drupal_8_theme_and_twig_sprint-rethemed-1778874-4.patch | 707 bytes | crasherzvir |
#2 | drupal_8_theme_and_twig_sprint-rethemed-1778874.patch | 1.1 KB | crasherzvir |
Comments
Comment #1
crasherzvir CreditAttribution: crasherzvir commentedstarted work
Comment #2
crasherzvir CreditAttribution: crasherzvir commentedComment #3
podarokwrong patch!
there is no file before in repository
Comment #4
crasherzvir CreditAttribution: crasherzvir commentedfixed
Comment #5
podarokshould use whitespace beetween {{ and variable
Comment #6
crasherzvir CreditAttribution: crasherzvir commentedfinal fix
Comment #7
podaroklooks good
Comment #8
podarokcommited
Comment #9
podarokcommited into 8.x branch
Comment #11
BerdirI am removing this template in #293318: Convert Aggregator feeds into entities because I think it's kinda useless and can be done using renderable arrays. Feel free to review/comment over there, need feedback from frontent people about how to improve that kinda long list of aggregator templates.
Comment #12
podarok#11 good to see here point to right comment with patch in the #293318: Convert Aggregator feeds into entities issue
and possibly converting to renderable would be good practice for minimizing template count during Twig conversion and it will be good example for developers