- π¬π§United Kingdom Alina Basarabeanu
Hi @dancbatista
We are getting a redirect from https://www.twitter.com to https://twitter.com/
Why was this issue closed without a fix? - π¬π§United Kingdom the_g_bomb
The patch added in #2 removes the forward slash needed at the end of the url. This should be added back in before merging.
- π¬π§United Kingdom the_g_bomb
FYI @dancbatista, there is no error message, but if you do any kind of SEO crawl report you will see that a 301 redirect happens.
When you use the module the link generated is https://www.twitter.com/mytwitter, when you follow the link end up at https://twitter.com/mytwitter, this happens via a 301 redirect.
Rather than relying on a redirect the link should be pointed at the final destination where possible. In this case that final destination is https://twitter.com/ not https://www.twitter.com/
The patch applies and if it can be updated to include the removed forward slash it will work well.
- @the_g_bomb opened merge request.
- π¬π§United Kingdom the_g_bomb
If you need to test with a patch please use:
https://git.drupalcode.org/project/social_media_links/-/merge_requests/2...