Problem/Motivation

These used to be services but aren't anymore.

Proposed resolution

Reword the docblock

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
remove_service_class_docblock-1.patch1.13 KBtimmillwood
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

timmillwood created an issue. See original summary.

timmillwood’s picture

timmillwood
Primary language English
Location 🏴󠁧󠁢󠁷󠁬󠁳󠁿 Wales, UK
CreditAttribution: timmillwood at Appnovation for Pfizer, Inc. commented
Status: Active » Needs review
markdorison’s picture

markdorison
he/him
CreditAttribution: markdorison at Chromatic commented
Status: Needs review » Reviewed & tested by the community

Looks good.

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Third and Grove commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.3.x and cherry-picked to 8.2.x. Thanks!

  • catch committed 45b845d on 8.3.x 
    Issue #2781179 by timmillwood: Remove service from class docblocks

  • catch committed 6c3ac1a on 8.2.x 
    Issue #2781179 by timmillwood: Remove service from class docblocks

(...

Status: Fixed » Closed (fixed)

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