fix: add favicon to resolve 404 on /favicon.ico (Closes #134) - #633
Open
astra-intelligence wants to merge 3 commits into
Open
astra-intelligence wants to merge 3 commits into
astra-intelligence wants to merge 3 commits into
Conversation
added 3 commits
September 29, 2026 14:22
- Added og:image:width (1200) and og:image:height (630) meta tags - Changed default OG image from small logo (next.jpeg) to proper 1200x630 social preview banner (/assets/img/og-default.jpg) - OG image features 'NextCommunity Developer Directory' branding with modern dark tech aesthetic and blue (#2563eb) accent theme Closes NextCommunity#499
Both index.njk and games.njk had frontmatter explicitly overriding the default OG image with next.jpeg. Updated to og-default.jpg so the new 1200x630 social preview banner is used across all pages.
- Creates 16x16 and 32x32 ICO favicon (retro terminal 'N' motif) - Adds 180x180 PNG for Apple Touch Icon - Adds favicon/apple-touch-icon link tags to base layout header Closes NextCommunity#134
astra-intelligence
requested review from
BaseMax and
jbampton
as code owners
September 30, 2026 07:55
 |
|
Overall Grade |
Security Reliability Complexity Hygiene |
Code Review Summary
| Analyzer | Status | Updated (UTC) | Details |
|---|---|---|---|
| JavaScript |  |
Sep 30, 2026 7:55a.m. | Review ↗ |
| Secrets | |
Sep 30, 2026 7:55a.m. | Review ↗ |
Important
AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.
This branch has not been deployed
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
1 participant
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.
Summary
Fixes the 404 error at
/favicon.icoby creating a proper favicon for the site and adding the necessary<link>tags in the base layout.Changes
base.njklayout:<link rel="icon" type="image/x-icon">for legacy browsers<link rel="icon" type="image/png">for modern browsers<link rel="apple-touch-icon">for iOS SafariCloses #134
🤖 PR generated by Adventure Agent (Astra Intelligence Labs)