Skip to content

feat(web): link to the other OpenCore sites from the footer - #27

Merged
leoisadev1 merged 1 commit into
mainfrom
opencore-sites-footer
Oct 23, 2026
Merged

leoisadev1 merged 1 commit into
mainfrom
opencore-sites-footer

Conversation

@leoisadev1

@leoisadev1 leoisadev1 commented Oct 23, 2026 •

Copy link
Copy Markdown
Member

Summary

Leo's public sites didn't link to each other. This adds a "More from OpenCore" group to the landing page footer, between the link columns and the bottom bar, with plain followed links to the other 13 OpenCore sites. The list lives in apps/web/src/lib/opencore-sites.ts, and every site's repo keeps an identical copy.

 <footer>
   existing footer content
+  More from OpenCore
+    <a href="https://social-sdk.dev">Social SDK  TypeScript SDK for social APIs</a>
+    …one link per site, this site's own domain left out
 </footer>

Evidence

Before (production) After (local build)
Before, desktop After, desktop
Before, mobile After, mobile
  • Before: the footer has no links to the other sites.
    After: a link check on the dev server's server-rendered / finds all 13 links with the exact hrefs and no nofollow, ugc or sponsored, and the site's own domain is left out. The links are in the server-rendered HTML, not added on the client.
  • oxfmt, oxlint and tsc --noEmit in apps/web pass.
  • Screenshots at 1440px and 390px with agent-browser.
  • Not checked yet: production. Each site's HTML gets the same link check after deploy.

Merge Danger

Door: two-way. Reverting the commit removes the links.

Blast radius: the footer markup and styles only. Docs sync: no docs page owns the landing footer, so there is nothing to update; flagging that gap here as AGENTS.md asks. Production deploys of the web app currently fail in CI because the repo has no VERCEL_TOKEN secret, so this ships with a manual vercel deploy --prebuilt --prod.

Created with Claude Opus 5.5 in Claude Code.


Devin Review

Note

Add "More from OpenCore" cross-site links to web footer

  • Adds a typed registry of 14 OpenCore sites (name, description, URL) plus otherOpenCoreSites(currentUrl), which filters out the supplied site, in opencore-sites.ts.
  • The footer in home-footer.tsx now renders a "More from OpenCore" section listing all sites except Amend itself. Existing footer content and bottom bar are unchanged.

Macroscope summarized eefdd3e.

Adds a "More from OpenCore" row to the landing footer, between the link
columns and the bottom bar. The links are plain followed links, kept in
apps/web/src/lib/opencore-sites.ts.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Devin Review found 1 potential issue.

Devin Review

@greptile-apps

greptile-apps Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

RetriggerView in GreptileConfidence Score: 5/5

[Low impact] The PR appears safe to merge; no actionable issues were found.

Summary

Adds “More from OpenCore” between the existing footer columns and bottom bar.

  • The amend.sh footer links visitors to the other OpenCore sites.

Reviews (1) · Last reviewed commit: "feat(web): link to the other OpenCore si..." · Reviewed by Greptile

@macroscopeapp

macroscopeapp Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

Approvability

Verdict: Approved at eefdd3e

Macroscope's review found this PR approvable — Adds a static, typed list of OpenCore sites and renders 13 plain external links in the landing-page footer, without changing existing flows or backend behavior. The scope is isolated to low-risk footer presentation and navigation.

You can add or adjust custom eligibility rules. Learn more.

@leoisadev1
leoisadev1 merged commit ccb3900 into main Oct 23, 2026
6 of 7 checks passed
@leoisadev1
leoisadev1 deleted the opencore-sites-footer branch October 23, 2026 20:55
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.

1 participant