Skip to content

fix: remove Telegram link from website footer - #2829

Open
chip-peanut-bot[bot] wants to merge 1 commit into
mainfrom
hotfix/remove-telegram-footer-link
Open

fix: remove Telegram link from website footer#2829
chip-peanut-bot[bot] wants to merge 1 commit into
mainfrom
hotfix/remove-telegram-footer-link

Conversation

@chip-peanut-bot

Copy link
Copy Markdown
Contributor

Replacement for #2826; clean hotfix branch from main.

What changed

  • removed the Telegram community link from the public website footer
  • removed the now-unused footer icon import
  • left X, GitHub, and all non-footer Telegram behavior unchanged

Verification

  • confirmed the PR contains one file with two deletions
  • confirmed the footer no longer contains the Telegram URL, label, or icon import

Co-authored-by: Aleks Balinda <aleks@peanut.me>
Co-authored-by: Chip <chip-bot@users.noreply.github.com>
@vercel

vercel Bot commented Aug 26, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
peanut-wallet Ready Ready Preview Aug 26, 2026 4:30pm

Request Review

@github-actions

Copy link
Copy Markdown
Contributor

Code-analysis diff

Painscore total: 7330.76 → 7330.7 (-0.06)
Findings: 0 net (+1 new, -1 resolved)

🆕 New findings (1)

  • medium high-mdd — src/components/LandingPage/Footer.tsx:55 — Footer: MDD 45.8 (uses across many lines from declarations)

✅ Resolved (1)

  • src/components/LandingPage/Footer.tsx:57 — Footer: MDD 45.8 (uses across many lines from declarations)

@github-actions

Copy link
Copy Markdown
Contributor

🧪 UI test report — ✅ all green

Suites

  • unit: 3479 ran, 0 failed, 0 skipped, 57.6s

📊 Coverage (unit)

metric %
statements 68.7%
branches 53.7%
functions 59.9%
lines 69.5%
⏱ 10 slowest test cases
time test
4.1s src/components/Card/share-asset/__tests__/shareAssetLayout.test.ts › never places two stickers in heavy overlap (broad seed sweep)
1.0s src/hooks/query/__tests__/user.test.tsx › does NOT clear a token that rotated mid-request (stale 401 racing a fresh login)
0.8s src/utils/__tests__/demo-api.test.ts › isDemoMode() is false when not running under Capacitor
0.6s src/utils/__tests__/crisp.test.ts › settles, and hands back a usable plugin, against a real-shaped plugin proxy
0.5s src/utils/__tests__/crisp.test.ts › configures once across repeated support opens
0.4s src/utils/__tests__/sentry.utils.test.ts › defaults to the client budget under a browser global
0.4s src/utils/__tests__/crisp.test.ts › retries configuration on the next open after a failure
0.4s src/utils/__tests__/crisp.test.ts › resets the native session on logout once support has been opened
0.4s src/components/Card/share-asset/__tests__/shareAssetLayout.test.ts › every sticker stays within canvas at any count
0.3s src/app/(mobile-ui)/withdraw/__tests__/withdraw-states.test.tsx › Bank withdrawal keeps the $1 minimum for sub-$1 amounts
📍 Inline annotations are in the **Unit test report** check above. Coverage artifact: `coverage-unit`. Generated by `.github/workflows/tests.yml`.

@chip-peanut-bot chip-peanut-bot Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Chip review: Review clean

No actionable defects found. The exact two-line deletion removes the Telegram footer entry and its now-unused icon import without changing the shared desktop/mobile social rendering or other Telegram behavior.

Checked clean

  • Confirmed the detached worktree HEAD, PR head/base metadata, and merge base match the supplied SHAs.
  • Reviewed the complete one-file diff and surrounding Footer implementation for correctness, security, adversarial failure paths, and slop.
  • Confirmed both desktop and mobile footer socials consume the same updated SOCIALS array and retain X and GitHub links.
  • Confirmed separate Telegram-sharing UI remains unchanged outside the footer.
  • CI at the exact head SHA is green for format, typecheck, unit, eslint, e2e, CodeQL, and preview deployment.

Second opinion skipped: openrouter-error.

Exact head: 4d393a7fa7a1 · Context: repo

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