Part of meta-issue #1856630: [Change notice] [META] Rename Views methods to core standards

CommentFileSizeAuthor
#1 drupal-get_render_tokens-2003268.patch5.2 KBdwieeb
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dwieeb’s picture

dwieeb CreditAttribution: dwieeb commented
Assigned: dwieeb » Unassigned
Status: Active » Needs review
FileSize
drupal-get_render_tokens-2003268.patch5.2 KB
SlayJay’s picture

SlayJay CreditAttribution: SlayJay commented
Status: Needs review » Reviewed & tested by the community

reviewed the function name changes, looks good.

Status: Reviewed & tested by the community » Needs work
Issue tags: -Novice, -VDC

The last submitted patch, drupal-get_render_tokens-2003268.patch, failed testing.

ThomasVI’s picture

ThomasVI CreditAttribution: ThomasVI commented
Status: Needs work » Needs review

#1: drupal-get_render_tokens-2003268.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, drupal-get_render_tokens-2003268.patch, failed testing.

oenie’s picture

oenie CreditAttribution: oenie commented
Status: Needs work » Needs review
Issue tags: +Novice, +VDC

#1: drupal-get_render_tokens-2003268.patch queued for re-testing.

If this retest passes, the patch is RTBC for me !

oenie’s picture

oenie CreditAttribution: oenie commented
Status: Needs review » Reviewed & tested by the community

Retest passed!

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Fixed

Committed be27377 and pushed to 8.x. Thanks!

Automatically closed -- issue fixed for 2 weeks with no activity.