Skip to content

feat(admin): Add dashboard analytics and shared tracking - #117

Merged
nfebe merged 4 commits into
devfrom
feat/admin-dashboard-update
Oct 24, 2026
Merged

nfebe merged 4 commits into
devfrom
feat/admin-dashboard-update

Conversation

@nfebe

@nfebe nfebe commented Oct 24, 2026

Copy link
Copy Markdown
Contributor

Add shared admin analytics and optional PostHog tracking. Keep shared design imports safe during server rendering.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Oct 24, 2026 •

Copy link
Copy Markdown

Deploying trakli-dev with  Cloudflare Pages  Cloudflare Pages

Latest commit: 77d1088
Status: ✅  Deploy successful!
Preview URL: https://4cde3d1d.trakli-dev.pages.dev
Branch Preview URL: https://feat-admin-dashboard-update.trakli-dev.pages.dev

View logs

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying webui with  Cloudflare Pages  Cloudflare Pages

Latest commit: 77d1088
Status: ✅  Deploy successful!
Preview URL: https://2ddff11a.webui-9fh.pages.dev
Branch Preview URL: https://feat-admin-dashboard-update.webui-9fh.pages.dev

View logs

@sourceant

sourceant Bot commented Oct 24, 2026

Copy link
Copy Markdown

Code Review Summary

The analytics configuration moves to nested runtime config (engagement, posthog) so NUXT_PUBLIC_* variables override it, and admin English messages from eloquent-admin-ui are merged into the i18n config. A patch-package patch guards window access in the shared design build, with a Node-environment test covering SSR import; the app's own engagement client plugin is deleted in favour of the upgraded package.

🚀 Key Improvements

  • Server-side imports of @whilesmart/design no longer throw on missing window, verified by a node-environment test.
  • Nested engagement and PostHog runtime config is overridable via environment variables.
  • Admin i18n strings ship with the app through eloquent-admin-ui messages merged into the en locale.

📉 Regressions

  • Client route-change page-view tracking is removed from this repo and now relies entirely on @whilesmart/engagement 0.3.0.

💡 Minor Suggestions

  • Review coverage is incomplete. Inspect the execution details before approving.

@sourceant sourceant Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review complete. See the overview comment for a summary.

@nfebe
nfebe merged commit 271fb92 into dev Oct 24, 2026
6 checks passed
@nfebe
nfebe deleted the feat/admin-dashboard-update branch October 24, 2026 06:42
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