Skip to content

chore: remove org logo from metadata (open graph) - #60

Merged
calebephrem merged 1 commit into
mainfrom
calebephrem-patch-1
Aug 28, 2026
Merged

chore: remove org logo from metadata (open graph)#60
calebephrem merged 1 commit into
mainfrom
calebephrem-patch-1

Conversation

@calebephrem

Copy link
Copy Markdown
Member

What does this PR do?

Removes organization logo from open graph

Closes

  • Closes [none]

Type of change(s)

  • Bug fix
  • New feature
  • Documentation update
  • Style / UI change
  • Refactor (no functional change)
  • Performance improvement
  • Other

Checklist

  • I've tested this change locally and it works as expected
  • bun run lint or npm run lint completes without warnings or errors
  • bun run build or npm run build completes successfully
  • My commit messages follow Conventional Commits (e.g. feat: ..., fix: ..., chore: ...)
  • I've updated relevant documentation (README, comments, etc.) if needed
  • I've removed any temporary debugging code (e.g. console.log)

@devhub-bot devhub-bot Bot added the chore maintenance, configs, formatting, dependencies label Aug 28, 2026
@beetle-ai

beetle-ai Bot commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Summary by Beetle

This PR removes the organization logo from the Open Graph metadata configuration. The change simplifies the metadata by eliminating a redundant image asset reference (the DevHub Organization Logo) from the social media preview metadata, keeping only the primary DevHub Community Banner image for Open Graph sharing.

📁 File Changes Summary (Consolidated across all commits):

File Status Changes Description
lib/metadata.ts Modified +0/-6 Removed the organization logo image object from the Open Graph images array in the metadata configuration. The logo entry (500x500px PNG from the org's GitHub assets) was deleted, leaving only the DevHub Community Banner as the primary Open Graph image.

Total Changes: 1 file changed, +0 additions, -6 deletions

🎯 Key Changes:

  • Removed Open Graph Image Entry: Deleted the organization logo image object from the metadata's Open Graph images array
  • Simplified Social Preview: The website now uses only the DevHub Community Banner (1200x630px) for social media previews instead of including both the banner and logo
  • Reduced Asset References: Eliminated the external GitHub asset reference (https://raw.githubusercontent.com/open-devhub/.github/main/assets/org_icon.png)

📊 Impact Assessment:

  • Security: ✅ No security implications. This is a metadata configuration change with no functional or authentication impact.
  • Performance: ✅ Minimal positive impact. Removing one image reference from the metadata reduces the payload size slightly and eliminates one external asset lookup during page rendering.
  • Maintainability: ✅ Improved. Simplifying the Open Graph configuration makes it easier to maintain and reduces complexity. Single image source is clearer than multiple image options.
  • Testing: ✅ Low testing impact. This change affects only metadata rendering and social media preview generation. Existing tests for metadata structure should still pass; however, any tests specifically validating the presence of the organization logo in Open Graph metadata would need updating.
⚙️ Settings

Severity Threshold: Medium — Balanced feedback — medium and high severity issues only.Change in Settings
Custom Rules: Define your own review rules — Set Custom Rules
PR Summary: Configure PR summary — Change in Settings

📖 User Guide
  • Once repos are connected, PR analysis is automatically enabled. You can disable analysis for this repo from beetleai.dev/analysis
  • Comment @beetle on any PR to start analysis manually
  • Comment @beetle stop to stop any ongoing analysis

Follow us: Beetle · X · LinkedIn

@beetle-ai

beetle-ai Bot commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

✅ You're good to merge this PR! No issues found. Great job!

Settings
⚙️ Settings

Severity Threshold: Medium — Balanced feedback — medium and high severity issues only.Change in Settings
Custom Rules: Define your own review rules — Set Custom Rules
PR Summary: Configure PR summary — Change in Settings

📖 User Guide
  • Once repos are connected, PR analysis is automatically enabled. You can disable analysis for this repo from beetleai.dev/analysis
  • Comment @beetle on any PR to start analysis manually
  • Comment @beetle stop to stop any ongoing analysis

@devhub-bot

devhub-bot Bot commented Aug 28, 2026

Copy link
Copy Markdown

Note

Linting checks passed successfully 🎉

All formatting and code quality checks are clean.

You're good to merge 🚀

@calebephrem
calebephrem merged commit a251b60 into main Aug 28, 2026
7 checks passed
@calebephrem
calebephrem deleted the calebephrem-patch-1 branch August 28, 2026 22:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore maintenance, configs, formatting, dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant