For URLs posted to Twitter, there's little point in shortening URLs; Twitter is just going to t.co them anyway (even if the URL we generate is shorter).

It's probably worth keeping support for Shorten module in so that we can shorten URLs going to Twitter-compatible services, but I think we could drop the builtin TinyURL shortening without any loss of functionality.

CommentFileSizeAuthor
#1 twitter_post-1816308-default_to_url_alias.patch1.49 KBxurizaemon

Comments

xurizaemon’s picture

xurizaemon
he/him
Location Ōtepoti, Aotearoa 🏝
commented
Status: Active » Needs review
StatusFileSize
new twitter_post-1816308-default_to_url_alias.patch1.49 KB

Maybe as a starter, let's just swap out !tinyurl for !url-alias in the default message?

xurizaemon’s picture

xurizaemon
he/him
Location Ōtepoti, Aotearoa 🏝
commented
Status: Needs review » Fixed

Committed in e1ab92e.

Status: Fixed » Closed (fixed)

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