Skip to content

docs(readme): "Made by" app links + Sponsor button#14

Merged
hamen merged 2 commits into
mainfrom
readme/made-by-apps
Jul 15, 2026
Merged

docs(readme): "Made by" app links + Sponsor button#14
hamen merged 2 commits into
mainfrom
readme/made-by-apps

Conversation

@hamen

@hamen hamen commented Jul 15, 2026

Copy link
Copy Markdown
Owner

Adds a Made by section at the end of the README linking all of Ivan's live app sites, and a .github/FUNDING.yml Sponsor button.

Why: the repo has a warm, high-trust dev audience. README/profile links are nofollow (no DR lift) but drive real referral traffic + installs, which is what ad/subscription-monetized apps need. UTM tags (utm_campaign=compose_skill) let GA4 attribute installs back to this repo.

Changes

  • README.md: new ## Made by section (10 app links + hub + X handle)
  • .github/FUNDING.yml: Sponsor button → GitHub Sponsors (github: [hamen]), per cross-review (the button must target a funding destination, not a commercial hub)

🤖 Generated with Claude Code

https://claude.ai/code/session_01BDaGsxPqiYywmcQLuFQRuH

hamen and others added 2 commits July 15, 2026 23:42
Add a "Made by" section linking all of Ivan's live app sites (with UTM
tags per repo) and a .github/FUNDING.yml Sponsor button pointing at the
hub. Drives referral traffic + installs from the repo's dev audience.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01BDaGsxPqiYywmcQLuFQRuH
Cross-review (Codex/Cursor/Antigravity) flagged that GitHub's Sponsor
button must target a funding destination, not a commercial app hub. Use
GitHub Sponsors (github: hamen), which is active for the account.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01BDaGsxPqiYywmcQLuFQRuH
@hamen

hamen commented Jul 15, 2026

Copy link
Copy Markdown
Owner Author
🟢 Codex review (automated cross-review)

Blocker

None.

Should-fix

None.

Nit

  • The PR body still says .github/FUNDING.yml points to ivanmorgillo.com; the latest commit correctly changed it to github: [hamen]. Update the description to match.

All added links resolve successfully; the change otherwise looks good.

via pr-review-relay · author: claude

@hamen

hamen commented Jul 15, 2026

Copy link
Copy Markdown
Owner Author
🔵 Cursor review (automated cross-review)

PR #14 Review — docs(readme): "Made by" app links + Sponsor button

Docs-only (+20 lines). I read the PR via gh, the full diff, and the touched files in the worktree. All 11 external URLs return HTTP 200; ivanmorgillo.com redirects to www.ivanmorgillo.com with UTMs preserved. github.com/sponsors/hamen also resolves. No code, CI, or security surface area is affected.


Blocker

None.


Should-fix

PR description is stale vs. the actual diff. The body says .github/FUNDING.yml points to the ivanmorgillo.com hub, but the file uses github: [hamen] (GitHub Sponsors). That’s the correct YAML for a Sponsor button and matches a live sponsors page — just update the PR text so reviewers aren’t misled. (Prior automated reviews that flagged a custom: portfolio URL appear to be reviewing an older revision.)


Nit

  • UTM parity: App and hub links all carry utm_source=github&utm_medium=readme&utm_campaign=compose_skill; the @hamen link does not. Add UTMs there only if you want X click attribution in GA4.
  • Maintenance: Ten hardcoded app entries will need manual README edits as the portfolio changes. Reasonable tradeoff if per-app README attribution matters; otherwise the hub link alone is easier to keep current.
  • Placement: ## Made by sits cleanly between the layout tree block and ## License — consistent with the README’s footer-style sections and non-intrusive for a maintainer-owned repo.

Verdict: Looks good — safe to merge. Only actionable item is aligning the PR description with github: [hamen] in FUNDING.yml.

via pr-review-relay · author: claude

@hamen

hamen commented Jul 15, 2026

Copy link
Copy Markdown
Owner Author
🟠 Antigravity review (automated cross-review)

Looks good — safe to merge.

via pr-review-relay · author: claude

@hamen
hamen merged commit f815c31 into main Jul 15, 2026
1 check passed
@hamen
hamen deleted the readme/made-by-apps branch July 15, 2026 22:10
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