Skip to content

Add Support for umami Analytics - #4200

Merged
mesudip merged 1 commit into
developfrom
feat/analytics
Aug 27, 2026
Merged

Add Support for umami Analytics#4200
mesudip merged 1 commit into
developfrom
feat/analytics

Conversation

@mesudip

@mesudip mesudip commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

List of changes

  • Remove Google Tag manager
  • Add Umami analytics. (dynamically injected only when running on production)

Design Decisions

  • Umami script was injected dynamically in the the index.html, preventing initialization during dev build
  • Nginx config is dynamically generated by entrypoint at runtime.
  • Umami traffic is directed via nginx container, that serves the frontend. This prevents cross origin requests from the frontend.
  • There are no regressions in the product features.

Checklist

  • My changes generate no new warnings
  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the changelog
  • I have added tests that prove my fix is effective or that my feature works

@mesudip
mesudip requested a review from kusssal August 25, 2026 08:07
@mesudip
mesudip requested a review from aaboyle878 as a code owner August 25, 2026 08:07

@kusssal kusssal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks Good

@mesudip
mesudip merged commit 1a25121 into develop Aug 27, 2026
8 of 9 checks passed
@mesudip
mesudip deleted the feat/analytics branch August 27, 2026 04:20
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.

2 participants