chore: add plausible.io #185
Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds Plausible.io analytics integration to the web application to enable privacy-friendly analytics tracking.
Changes:
- Added Plausible.io analytics script and initialization code to the HTML head section
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
SgtPooki
force-pushed
the
fix/add-plausible-io
branch
from
89d85ca to
abf1ffa
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| - **Required**: No | ||
| - **Default**: Empty (Plausible disabled) | ||
| - **Location**: `apps/web/.env` | ||
|
|
||
| **Role**: Enables Plausible analytics for the web frontend when set. The value should match the Plausible site domain |
There was a problem hiding this comment.
This section says the variable’s location is apps/web/.env, but Vite VITE_* values are baked in at build time. In this repo the web Docker build explicitly wires build-time env via ARG/ENV (see apps/web/Dockerfile for VITE_API_BASE_URL), so readers may incorrectly assume they can toggle Plausible at runtime by changing container env. Please clarify whether VITE_PLAUSIBLE_DATA_DOMAIN must be provided at build time (and, if so, how it’s set in Docker/K8s deployments).
fixes #180
plausible is not enabled in local dev.
plausible is not enabled unless someone explicitly sets
VITE_PLAUSIBLE_DATA_DOMAINenv var