Skip to content

fix:Use FORRT banner as default social sharing image#696

Merged
LukasWallrich merged 4 commits intomasterfrom
scpreview
Mar 12, 2026
Merged

fix:Use FORRT banner as default social sharing image#696
LukasWallrich merged 4 commits intomasterfrom
scpreview

Conversation

@richarddushime
Copy link
Contributor

@richarddushime richarddushime commented Mar 9, 2026

Description

When the Steering Committee page was shared on social platforms, previews showed the first image on the page instead of FORRT logo.

Cause: The default sharing image was set to FORRT_banner.svg. Many platforms don’t use SVG for link previews and fall back to the first in the HTML, which on that page was a member profile.

Change: Set the site-wide default sharing image to img/FORRTbanner.webp in config/_default/params.toml.

Thanks to Adira and flavio

Type of Change

  • Bug fix

Testing

Checklist

  • Self-reviewed my changes
  • Verified links and formatting are correct
  • No new warnings or errors

Notes

@richarddushime richarddushime requested a review from a team as a code owner March 9, 2026 22:38
@github-actions
Copy link
Contributor

github-actions bot commented Mar 9, 2026

👍 All image files/references (if any) are in webp format, in line with our policy.

@forrtproject forrtproject deleted a comment from github-actions bot Mar 9, 2026
@LukasWallrich
Copy link
Contributor

LukasWallrich commented Mar 9, 2026

Staging Deployment Status

This PR has been successfully deployed to staging as part of an aggregated deployment.

Deployed at: 2026-03-12 11:04:02 UTC
Staging URL: https://staging.forrt.org

The staging site shows the combined state of all compatible open PRs.

@richarddushime
Copy link
Contributor Author

richarddushime commented Mar 9, 2026

@forrtproject forrtproject deleted a comment from github-actions bot Mar 9, 2026
@forrtproject forrtproject deleted a comment from github-actions bot Mar 9, 2026
@LukasWallrich
Copy link
Contributor

Thanks @richarddushime. This is obviously a very boring image, so ok as a fallback, but can we document how to specify another image (maybe in your new contributing instructions?)

@github-actions
Copy link
Contributor

📝 Spell Check Results

Found 8 potential spelling issue(s) in this PR:

📄 content/clusters/cluster7.md

Line Issue
130 Tham ==> Than
171 Tham ==> Than
201 Tham ==> Than

📄 content/educators-corner/004-Teaching-why-how-replication/index.md

Line Issue
94 pre-selected ==> preselected

📄 content/glossary/english/statistical_significance.md

Line Issue
5 pre-selected ==> preselected

📄 content/pedagogies/003-CREP/index.md

Line Issue
86 pre-selected ==> preselected
126 pre-selected ==> preselected
134 pre-selected ==> preselected

ℹ️ How to address these issues:

  1. Fix the typo: If it's a genuine typo, please correct it.
  2. Add to whitelist: If it's a valid word (e.g., a name, technical term), add it to .codespell-ignore.txt
  3. False positive: If this is a false positive, please report it in the PR comments.

🤖 This check was performed by codespell

Copy link
Contributor

@LukasWallrich LukasWallrich left a comment

Choose a reason for hiding this comment

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

Looks good

@LukasWallrich LukasWallrich merged commit aae474c into master Mar 12, 2026
3 checks passed
@LukasWallrich LukasWallrich deleted the scpreview branch March 12, 2026 13:14
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