The trusted browser setup plugin contains invalid placeholders that start with !, which were removed from Drupal 8.

CommentFileSizeAuthor
#4 2913905-3-invalid-placeholder.patch5.21 KBsanduhrs
#2 2913905-invalid-placeholder.patch2.33 KBdavisben
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

davisben created an issue. See original summary.

davisben’s picture

davisben
Primary language English
Location NH, USA
CreditAttribution: davisben commented
FileSize
2913905-invalid-placeholder.patch2.33 KB
davisben’s picture

davisben
Primary language English
Location NH, USA
CreditAttribution: davisben commented
Status: Active » Needs review
sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
CreditAttribution: sanduhrs at erdfisch commented
Status: Needs review » Reviewed & tested by the community
FileSize
2913905-3-invalid-placeholder.patch5.21 KB

Added some minor code style updates.
Other than that this is fine and RTBC.

nerdstein’s picture

nerdstein
he/him
Primary language English
Location United States
CreditAttribution: nerdstein as a volunteer and at CivicActions commented
Status: Reviewed & tested by the community » Fixed

This also looks good. thank you.

Status: Fixed » Closed (fixed)

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