CommentFileSizeAuthor
#4 interdiff-3069050-2-4.txt941 bytesvoleger
#4 3069050-4.patch2.14 KBvoleger
#2 3069050-2.patch2.14 KBvoleger

Comments

mikelutz created an issue. See original summary.

voleger’s picture

voleger
he/his
Primary language Ukrainian
Location Ukraine, Rivne
commented
Status: Active » Needs review
StatusFileSize
new 3069050-2.patch2.14 KB

Here the patch with proper deprecation.

Status: Needs review » Needs work

The last submitted patch, 2: 3069050-2.patch, failed testing. View results

voleger’s picture

voleger
he/his
Primary language Ukrainian
Location Ukraine, Rivne
commented
Status: Needs work » Needs review
StatusFileSize
new 3069050-4.patch2.14 KB
new interdiff-3069050-2-4.txt941 bytes

Removed slash from an expected message

voleger’s picture

voleger
he/his
Primary language Ukrainian
Location Ukraine, Rivne
commented

Updated references to this issue in https://www.drupal.org/node/2783545

mikelutz’s picture

mikelutz
He/Him
Primary language English
Location Michigan, USA
commented
Status: Needs review » Reviewed & tested by the community

Good, the legacy test was split off into its own test so we don't lose any test coverage, and according to drupal-check, that was the only place this was used.

alexpott’s picture

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

Committed 7237ed3 and pushed to 8.8.x. Thanks!

  • alexpott committed 7237ed3 on 8.8.x 
    Issue #3069050 by voleger, mikelutz: Properly deprecate...

Status: Fixed » Closed (fixed)

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