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

CommentFileSizeAuthor
#2 drupal-camelcase-replace-2002454-7450966.patch1.15 KBbaldwinlouie

Comments

baldwinlouie’s picture

baldwinlouie commented
Assigned: Unassigned » baldwinlouie

taking this: baldwinlouie and sillygwailo

baldwinlouie’s picture

baldwinlouie commented
Status: Active » Needs review
StatusFileSize
new drupal-camelcase-replace-2002454-7450966.patch1.15 KB

attaching patch

Status: Needs review » Needs work
Issue tags: -Novice, -VDC

The last submitted patch, drupal-camelcase-replace-2002454-7450966.patch, failed testing.

sillygwailo’s picture

sillygwailo
Location Toronto, ON
commented
Status: Needs work » Needs review

#2: drupal-camelcase-replace-2002454-7450966.patch queued for re-testing.

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

The last submitted patch, drupal-camelcase-replace-2002454-7450966.patch, failed testing.

heddn’s picture

heddn
Primary language English
Location Nicaragua
commented
Status: Needs work » Needs review
heddn’s picture

heddn
Primary language English
Location Nicaragua
commented

Looks good. Once test bots come back, this is RTBC.

aspilicious’s picture

aspilicious commented

#2: drupal-camelcase-replace-2002454-7450966.patch queued for re-testing.

dawehner’s picture

dawehner
Primary language German
commented
Status: Needs review » Reviewed & tested by the community

Still applies and code looks fine!

dawehner’s picture

dawehner
Primary language German
commented

Still applies and code looks fine!

alexpott’s picture

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

Committed c7ebc94 and pushed to 8.x. Thanks!

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