Added opengraph and twitter meta tags to index.html#1387
Open
KanakReshi wants to merge 3 commits into
Open
Conversation
|
|
github-actions
Bot
added
new contributor
KanakReshi
changed the title
Haz3-jolt
requested changes
Jun 7, 2026
Haz3-jolt
left a comment
Haz3-jolt
left a comment
Contributor
There was a problem hiding this comment.
Please sign CLA as well
| <meta name="twitter:card" content="summary_large_image" /> | ||
| <meta name="twitter:title" content="Observal" /> | ||
| <meta name="twitter:description" content="Agent registry with built-in observability" /> | ||
| <meta name="twitter:image" content="/og-image.png" /> |
Contributor
There was a problem hiding this comment.
Link the existing logo do not create your own
|
|
||
| await db.commit() | ||
| await db.refresh(listing) | ||
| if listing.status == ListingStatus.pending or listing.status == ListingStatus.rejected: |
Contributor
There was a problem hiding this comment.
Why this unrelated change??
Author
There was a problem hiding this comment.
This was done by me few days ago , I found a dead branch so i changed it , My bad
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
KanakReshi
force-pushed
the
add-opengraph-and-twitter-meta-tags
branch
from
06cb0b7 to
a1362e4
Compare
Haz3-jolt
requested changes
Jun 7, 2026
|
|
||
|
|
||
| # --- Version sub-routes --- | ||
| router.include_router(create_version_router("mcp", McpListing, McpVersion)) |
Contributor
There was a problem hiding this comment.
Your net diff is 0; what are you doing? Don't touch. what is not in scope
Contributor
|
Squash into one commit and add screenshots |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose / Description
web/index.htmlonly has a basic<title>and<meta name="description">but no OpenGraph or Twitter Card meta tags. When an Observal link is shared on Slack, Discord, Twitter/X, or LinkedIn, it renders as a bare URL with no rich preview (no title card, no description, no image). This PR adds the necessary meta tags so shared links display a professional preview card.Fixes
Approach
Added OpenGraph (
og:title,og:description,og:image,og:url,og:type) and Twitter Card (twitter:card,twitter:title,twitter:description,twitter:image) meta tags toweb/index.html. Usedsummary_large_imagefor the Twitter card type to maximize preview visibility. All tags use the existing title ("Observal") and description ("Agent registry with built-in observability") already present in the document, keeping values consistent and DRY. Image references point to/og-image.pnginweb/public/. A dedicated 1200×630 branded OG image was added for optimal social preview rendering.How Has This Been Tested?
index.htmlis valid HTML with correct tag placement inside<head>./og-image.pngresolves correctly fromweb/public/.Learning (optional, can help others)
summary_large_imageprovides the largest preview card on Twitter/X, making it the best choice for branding pages.Checklist