Skip to content

Dev#54

Merged
NeedToUpdate merged 4 commits into
mainfrom
dev
Jul 13, 2026
Merged

Dev#54
NeedToUpdate merged 4 commits into
mainfrom
dev

Conversation

@NeedToUpdate

Copy link
Copy Markdown
Owner

No description provided.

NeedToUpdate and others added 4 commits July 12, 2026 15:49
Bing's URL Inspection flagged the empty alt on these images. They
were empty deliberately (title already shown as adjacent text, so a
screen reader shouldn't announce it twice), but a descriptive alt is
the more standard, dual-purpose choice here and clears the notice.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
fix: use post title as alt text on recommendation thumbnails
Wires in posthog-js through Next.js's instrumentation-client.ts (no
provider/pageview boilerplate needed with the current SDK defaults).
Events route through a new /relay/* CloudFront behavior straight to
PostHog's own origin — a CloudFront Function strips the prefix, no
Lambda invocation involved — so requests are same-origin and don't
get dropped by ad/tracker blockers the way a direct posthog.com call
would. Dev and prod use separate PostHog projects, tokens injected
per-stage via GitHub Actions secrets at build time.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
feat: add PostHog analytics via a same-origin CloudFront proxy
@NeedToUpdate NeedToUpdate merged commit fa552df into main Jul 13, 2026
2 checks passed
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