Skip to content

Conversation

@Inglan
Copy link
Member

@Inglan Inglan commented Sep 18, 2025

Summary by CodeRabbit

  • New Features

    • Tracker dialog now includes a “Technical info” section that indicates when our services are being blocked, helping users diagnose issues.
    • Updated notice text for clearer guidance on allowing trackers/ad blockers for this site.
  • Style

    • Improved spacing in the tracker dialog for better readability.
  • Chores

    • Updated analytics service endpoints to a new host; no changes to user-facing behavior.

@Inglan Inglan linked an issue Sep 18, 2025 that may be closed by this pull request
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Sep 18, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

Redirected PostHog analytics endpoints to https://hog.edutools.ingo.au and updated the exception autocapture script URL accordingly. Adjusted tracker dialog text/spacing and added a technical info details block indicating the proxy host is being blocked.

Changes

Cohort / File(s) Summary
Analytics endpoint redirect
src/lib/analytics.ts
Updated PostHog api_host and exception-autocapture script URL to https://hog.edutools.ingo.au; no API or logic changes.
Tracker dialog copy and details
src/lib/components/tracker-dialog.svelte
Revised wording, adjusted line breaks, and added a details section noting blocking of hog.edutools.ingo.au.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  participant App
  participant Browser
  participant PostHog as PostHog SDK
  participant HogProxy as hog.edutools.ingo.au

  note over App,Browser: App startup
  App->>PostHog: posthog.init({ api_host: hog.edutools.ingo.au })
  PostHog-->>HogProxy: Send analytics events
  HogProxy-->>PostHog: Responses
  PostHog-->>App: Init complete

  note over App,HogProxy: Exception autocapture script
  App->>Browser: fetch https://hog.edutools.ingo.au/static/exception-autocapture.js
  alt Blocked by tracker/ad blocker
    Browser--x App: Fetch blocked/error
    App->>App: Trigger tracker dialog
  else Loaded
    Browser-->>App: Script loaded
  end
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related issues

Possibly related PRs

Suggested reviewers

  • nathblade16
✨ Finishing touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch 505-use-proxy-for-posthog

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ef07527 and b1f1860.

📒 Files selected for processing (2)
  • src/lib/analytics.ts (2 hunks)
  • src/lib/components/tracker-dialog.svelte (1 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Inglan Inglan changed the title use a proxy for posthog analytics use a reverse proxy for posthog analytics Sep 18, 2025
@Inglan Inglan merged commit e5fcf5b into main Sep 18, 2025
3 of 4 checks passed
@Inglan Inglan deleted the 505-use-proxy-for-posthog branch September 18, 2025 06:14
@coderabbitai coderabbitai bot mentioned this pull request Sep 24, 2025
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.

Use proxy for posthog

2 participants