From dbbe8d2a368fb63ca56a71bde51d287454fdb3f1 Mon Sep 17 00:00:00 2001 From: Pasindu Fernando <116358471+Pasindufdo98@users.noreply.github.com> Date: Sun, 30 Mar 2025 22:01:05 +1100 Subject: [PATCH 1/3] chroe: task/analytics component review --- docs/OnTrack/Front End Migration/Migration/task_analytics.md | 0 1 file changed, 0 insertions(+), 0 deletions(-) create mode 100644 docs/OnTrack/Front End Migration/Migration/task_analytics.md diff --git a/docs/OnTrack/Front End Migration/Migration/task_analytics.md b/docs/OnTrack/Front End Migration/Migration/task_analytics.md new file mode 100644 index 000000000..e69de29bb From 011194814e367813140071c2e4077e3c90e49105 Mon Sep 17 00:00:00 2001 From: Pasindu Fernando <116358471+Pasindufdo98@users.noreply.github.com> Date: Sun, 18 May 2025 13:14:58 +1000 Subject: [PATCH 2/3] docs: re-add task_analytics.md for analytics component review --- .../Migration/task_analytics.md | 67 +++++++++++++++++++ 1 file changed, 67 insertions(+) diff --git a/docs/OnTrack/Front End Migration/Migration/task_analytics.md b/docs/OnTrack/Front End Migration/Migration/task_analytics.md index e69de29bb..318c8a142 100644 --- a/docs/OnTrack/Front End Migration/Migration/task_analytics.md +++ b/docs/OnTrack/Front End Migration/Migration/task_analytics.md @@ -0,0 +1,67 @@ +# OnTrack Component Review + +## Team Member Name + +### Pasindu Fernando + +### Student ID : +s224263102 + +## Component Name +Analytics Configuration (virtual page views) + +### Files in this Component +analytics.coffee (to be removed) + +## Description + +The analytics.coffee file was previously responsible for configuring Google Analytics tracking in the AngularJS version of the application. Its key functionality included disabling automatic route change tracking by setting ``` $analyticsProvider.virtualPageviews(false) ```, preventing virtual pageviews from being recorded. + +## Why Removal is Required + +It is no longer functional due to the migration to Angular 17. + +All analytics functionality has been migrated to Google Analytics 4 (GA4), which is fully managed through gtag.js in index.html. + +Virtual pageviews and event tracking are now handled explicitly using Angular’s Router and manual event tracking with gtag(). + + +# How Has This Been Tested? + +Monitoring Network Activity (Before and After Removing analytics.coffee): there is no change in the network activity ( Browser developer tools-> network tab-> filter the keyword "google"), before and after removing analytics.coffee since tracking is now fully handled by gtag.js in index.html. + +Modifying virtualPageviews to True/False : No visible change in pageview tracking behavior, indicating that analytics.coffee is no longer controlling analytics. + +## Screenshots +before commenting analytics.coffee file and depenency in config.coffee + +![before commenting analytics.coffee file and depenency in config.coffee](https://github.com/user-attachments/assets/6e10e0c3-0962-4cad-9988-b8d08d54b4a6) + +After commenting analytics.coffee file and depenency in config.coffee +![after commenting analytics.coffee file and ddepenency in config.coffee](https://github.com/user-attachments/assets/176f026b-806e-4b21-bcc5-8ab493f93fc9) + + +## Testing Checklist: + +- [x] Tested in latest Chrome +- [ ] Tested in latest Safari +- [x] Tested in latest Firefox + +# Checklist: + +- [x] My code follows the style guidelines of this project +- [x] I have performed a self-review of my own code +- [ ] I have commented my code in hard-to-understand areas +- [ ] I have made corresponding changes to the documentation +- [x] My changes generate no new warnings +- [ ] I have requested a review from ... on the Pull Request + +## Component Migration Plan: + +1. Remove the File:analytics.coffee + + +2. Clean Up Dependencies: + Remove doubtfire.config.analytics from config.coffee. + Delete any references to angulartics or $analyticsProvider in doubtfire-angularjs.module.ts. + From a0905e59172a7fb087137ea848f5f109964fc552 Mon Sep 17 00:00:00 2001 From: Pasindu Fernando <116358471+Pasindufdo98@users.noreply.github.com> Date: Sun, 18 May 2025 14:42:39 +1000 Subject: [PATCH 3/3] Update task_analytics.md --- .../Migration/task_analytics.md | 22 ------------------- 1 file changed, 22 deletions(-) diff --git a/docs/OnTrack/Front End Migration/Migration/task_analytics.md b/docs/OnTrack/Front End Migration/Migration/task_analytics.md index 318c8a142..91a150517 100644 --- a/docs/OnTrack/Front End Migration/Migration/task_analytics.md +++ b/docs/OnTrack/Front End Migration/Migration/task_analytics.md @@ -26,35 +26,13 @@ All analytics functionality has been migrated to Google Analytics 4 (GA4), which Virtual pageviews and event tracking are now handled explicitly using Angular’s Router and manual event tracking with gtag(). -# How Has This Been Tested? - -Monitoring Network Activity (Before and After Removing analytics.coffee): there is no change in the network activity ( Browser developer tools-> network tab-> filter the keyword "google"), before and after removing analytics.coffee since tracking is now fully handled by gtag.js in index.html. - -Modifying virtualPageviews to True/False : No visible change in pageview tracking behavior, indicating that analytics.coffee is no longer controlling analytics. ## Screenshots before commenting analytics.coffee file and depenency in config.coffee ![before commenting analytics.coffee file and depenency in config.coffee](https://github.com/user-attachments/assets/6e10e0c3-0962-4cad-9988-b8d08d54b4a6) -After commenting analytics.coffee file and depenency in config.coffee -![after commenting analytics.coffee file and ddepenency in config.coffee](https://github.com/user-attachments/assets/176f026b-806e-4b21-bcc5-8ab493f93fc9) - - -## Testing Checklist: - -- [x] Tested in latest Chrome -- [ ] Tested in latest Safari -- [x] Tested in latest Firefox - -# Checklist: -- [x] My code follows the style guidelines of this project -- [x] I have performed a self-review of my own code -- [ ] I have commented my code in hard-to-understand areas -- [ ] I have made corresponding changes to the documentation -- [x] My changes generate no new warnings -- [ ] I have requested a review from ... on the Pull Request ## Component Migration Plan: