Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Social Media Preview Image for Ruby LSP Docs #2606

Merged
merged 1 commit into from
Sep 24, 2024

Conversation

alexcrocha
Copy link
Contributor

@alexcrocha alexcrocha commented Sep 24, 2024

Addresses https://github.com/Shopify/team-ruby-dx/issues/1272

This PR adds social media previews for the Ruby LSP documentation by setting a default image in Jekyll's configuration file.

Motivation

Currently, there is no image preview when sharing the docs on social media. By adding a default image, we can enhance the appearance and engagement of shared links.

Implementation

Added the Ruby LSP icon as the default image to jekyll/_config.yml. The jekyll-seo-tag dependency, which is already included in the just-the-docs theme, was used to handle this. I followed the instructions from the jekyll-seo-tag documentation to set up the default image.

@alexcrocha alexcrocha added documentation Improvements or additions to documentation chore Chore task labels Sep 24, 2024
@alexcrocha alexcrocha requested a review from a team as a code owner September 24, 2024 00:39
@alexcrocha alexcrocha requested review from andyw8 and st0012 September 24, 2024 00:39
Copy link
Contributor

@andyw8 andyw8 left a comment

Choose a reason for hiding this comment

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

I'm not sure of a good way to test this when not on production, so let's ship it since it's low risk.

@andyw8 andyw8 merged commit d037674 into main Sep 24, 2024
38 checks passed
@andyw8 andyw8 deleted the ar/social-media-preview branch September 24, 2024 13:54
@andyw8
Copy link
Contributor

andyw8 commented Sep 24, 2024

Once it deploys I will run it against https://socialsharepreview.com/

@andyw8
Copy link
Contributor

andyw8 commented Sep 24, 2024

Before:

Screenshot 2024-09-24 at 9 55 15 AM Screenshot 2024-09-24 at 9 55 10 AM

@andyw8
Copy link
Contributor

andyw8 commented Sep 24, 2024

Not fully working yet, I commented on the internal issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Chore task documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants