[GH-3412] feat: add opengraph tags to website #3413
Conversation
SethFalco
requested review from
sbishel and
chenilim
and removed request for
a team
|
Hello @SethFalco, Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here. |
|
Thanks @SethFalco! Much appreciated! cc @justinegeffen, @emdecr - I haven't used opengraph tags before, but these LGTM. Please review. |
There was a problem hiding this comment.
Thanks for the addition @SethFalco!
@chenilim I'll ask around the team as well to see if they have any suggestions for copy or image changes. Will make another PR with adjustments, if anything. But these changes all look good to me 👍
|
This PR has been automatically labelled "stale" because it hasn't had recent activity. |
|
The copy looks good to me, thank you! |
justinegeffen
added
2. Editor Review
and removed
Lifecycle/1:stale
2. Editor Review
labels
|
@wuwinson, can we merge this? :) |
|
This PR has been automatically labelled "stale" because it hasn't had recent activity. |
Summary
Adds OpenGraph tags to the website so it looks more attractive when shared in instant messengers or social media.
More information in the GitHub issue.
You're welcome to suggest changing any particular tag that was set, I just chose what I'd think are sensible values. A better image could be used, but thought I'd leave that one up to you rather, I opted to only pick from what static assets are already being served.
This does do a minor change on the side, if the
title(tab name) doesn't haveFocalboardin it, it appends| Focalboardto it. (Similar to the behavior you have on the Mattermost website.)For example, the landing page is still:
Focalboard: Open source alternative to Trello, Asana, and NotionBut most other pages went from a format like:
Personal Edition→Personal Edition | FocalboardTicket Link
Previews