This the last commit removed the http but left the colon, so links go to page not found and not twitter.

$post->full_username = '://twitter.com/' . $post->username;

CommentFileSizeAuthor
#2 remove-colon-from-twitter-url-3097180-1.patch621 bytesdk-massive

Comments

devitate created an issue. See original summary.

dk-massive’s picture

dk-massive commented
StatusFileSize
new remove-colon-from-twitter-url-3097180-1.patch621 bytes

Here is a patch removing that colon from the Twitter url.

dk-massive’s picture

dk-massive commented
Status: Active » Needs review
tyeth’s picture

tyeth commented

Looks good having checked for usage (href in template). Haven't got twitter setup yet so can't test. Will check back in...

  • Hemangi Gokhale committed 5d9843a on 8.x-1.x 
    Issue #3097180 by dk-massive: This colon is misplaced '://twitter.com/'
hemangi.gokhale’s picture

hemangi.gokhale
she/her/hers
Primary language English
Location Portland, 🇺🇸
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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