While evaluating this module, I noticed that on .info file the project name is in lower case and should be in title case to be consistent with the rest of the project names on the page. Patch to follow. Thanks!

CommentFileSizeAuthor
#3 after.png7.26 KBvolkswagenchick
#3 before.png10.39 KBvolkswagenchick
#2 remote-stream-wrapper-capitalization-2990487-2.patch347 byteslegaudinier
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

legaudinier created an issue. See original summary.

legaudinier’s picture

legaudinier CreditAttribution: legaudinier at Hook 42 commented
Assigned: legaudinier » Unassigned
Status: Active » Needs review
FileSize
remote-stream-wrapper-capitalization-2990487-2.patch347 bytes

I uploaded a patch that fixes the capitalization issue in the .info.yml file. Thanks!

volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
CreditAttribution: volkswagenchick at Hook 42 commented
Status: Needs review » Reviewed & tested by the community
FileSize
before.png10.39 KB
after.png7.26 KB

Patch applied using simplytest.me and the name of the module is now capitalized on the extend page.

Screenshots attached. Marking RTBC

  • Dave Reid committed 37c2a5c on 7.x-1.x authored by legaudinier 
    Issue #2990487 by legaudinier, volkswagenchick: Capitalize module name.

  • Dave Reid committed 03bbde3 on 8.x-1.x authored by legaudinier 
    Issue #2990487 by legaudinier, volkswagenchick: Capitalize module name.
Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid as a volunteer and at Lullabot commented
Status: Reviewed & tested by the community » Fixed

Thanks, fixed in 8.x-1.x and 7.x-1.x.

Status: Fixed » Closed (fixed)

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

ciss’s picture

ciss CreditAttribution: ciss at yousign GmbH commented

@Dave Reid Would it make sense to change the drupal.org project title accordingly?