Skip to content

Add CrawlProof ad units#130

Closed
crawlproof[bot] wants to merge 1 commit into
masterfrom
crawlproof/install-ad-embed-aa961cc7-mron6jg9
Closed

Add CrawlProof ad units#130
crawlproof[bot] wants to merge 1 commit into
masterfrom
crawlproof/install-ad-embed-aa961cc7-mron6jg9

Conversation

@crawlproof

@crawlproof crawlproof Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

This PR adds the CrawlProof ad units so this site can show network ads and earn crypto for clicks.

  • Slot: aa961cc7-96e6-4f77-b065-e962cd7bce6f
  • Sizes: banner_300x250, banner_728x90, banner_320x50, text_link
  • Injected all sizes into src/app/layout.tsx before </body>. Move or delete any you don't want — empty units simply don't render.

Each unit renders inside a sandboxed iframe and never blocks page load. Manage the slot at https://crawlproof.com/ads/slots

@github-actions

Copy link
Copy Markdown

vu1nz Security Review

0 finding(s) in PR #?

No security issues found.

@ralyodio

Copy link
Copy Markdown
Contributor

Superseded by in-content, logged-out-only ad placement on master (AdBanner/AdRectangle across /dht, /torrents, /blog + their detail pages). The ad.js loader is now global, so this PR's below-footer divs would double-load it and are redundant.

@ralyodio ralyodio closed this Jul 17, 2026
@ralyodio
ralyodio deleted the crawlproof/install-ad-embed-aa961cc7-mron6jg9 branch July 17, 2026 08:13
ralyodio added a commit that referenced this pull request Jul 17, 2026
Add an auth-gated <AdUnit>/<AdBanner>/<AdRectangle> component (renders only
for logged-out visitors) and place units in-content across /dht, /torrents,
and /blog lists (728x90 leaderboard / 320x50 mobile) and their detail pages
(300x250 rectangle). Load crawlproof ad.js globally in the root layout and
trigger scan() on mount since the loader scans once and does not observe DOM
mutations. Replaces PR #130's below-footer, ungated ad divs.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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