Skip to content

Conversation

@BramSuurdje
Copy link
Member

✍️ Description

This pull request updates the analytics integration in the frontend application. The main changes involve switching the analytics script to use the new API endpoint and updating the analytics token, along with a minor import adjustment to support the new script loading method.

Analytics integration updates:

  • Replaced the direct <script> tag for analytics in the RootLayout component with the Next.js <Script> component, loading the script from the /api/script.js endpoint and updating the attribute from data-website-id to data-site-id. The script now loads after the page becomes interactive. (frontend/src/app/layout.tsx)
  • Updated the analytics token from "aefee1b9-2a12-4ac2-9d82-a63113edc62e" to "f9eee289f931" in the analytics config object. (frontend/src/config/site-config.tsx)

Codebase maintenance:

  • Added the next/script import to support usage of the <Script> component. (frontend/src/app/layout.tsx)

🔗 Related PR / Issue

Link: #

✅ Prerequisites (X in brackets)

  • Self-review completed – Code follows project standards.
  • Tested thoroughly – Changes work as expected.
  • No security risks – No hardcoded secrets, unnecessary privilege escalations, or permission issues.

🛠️ Type of Change (X in brackets)

  • 🐞 Bug fix – Resolves an issue without breaking functionality.
  • New feature – Adds new, non-breaking functionality.
  • 💥 Breaking change – Alters existing functionality in a way that may require updates.
  • 🆕 New script – A fully functional and tested script or script set.
  • 🌍 Website update – Changes to website-related JSON files or metadata.
  • 🔧 Refactoring / Code Cleanup – Improves readability or maintainability without changing functionality.
  • 📝 Documentation update – Changes to README, AppName.md, CONTRIBUTING.md, or other docs.

@BramSuurdje BramSuurdje requested a review from a team as a code owner November 12, 2025 00:11
@github-actions github-actions bot added the website A change to the website label Nov 12, 2025
@MickLesk MickLesk merged commit 17354ca into main Nov 12, 2025
4 checks passed
@MickLesk MickLesk deleted the chage-umami-to-rybbit branch November 13, 2025 07:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

website A change to the website

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants