Skip to content

Conversation

@lmammino
Copy link
Contributor

@lmammino lmammino commented Mar 18, 2023

Missing the protocol might lead crawlers to think this is a relative URL. For example the FullStack bulletin crawler picked up an article URL as follows:

Screenshot 2023-03-18 at 10 59 17

This might also break previews for when articles are shared on social websites such as Twitter, Linkedin and Facebook.

For instance, Linkedin picks this up as HTTP:

Screenshot 2023-03-18 at 11 14 20

While Twitter doesn't seem to pick up the preview picture (not necessarily related but worth investigating too after this is fixed):

Screenshot 2023-03-18 at 11 15 02

@vercel
Copy link

vercel bot commented Mar 18, 2023

@lmammino is attempting to deploy a commit to the Codú Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Contributor

@NiallJoeMaher NiallJoeMaher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this! Points out the broken line but it's the host that is my issue. I'll fix this later. ❤️

Copy link
Contributor

@NiallJoeMaher NiallJoeMaher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nope you were right! This is what I get for reviewing code while in the gym 😂 Thanks!

@vercel
Copy link

vercel bot commented Mar 18, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
codu ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Mar 18, 2023 at 9:00PM (UTC)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants