Skip to content

Conversation

@zara1504
Copy link

@zara1504 zara1504 commented May 11, 2025

Analytics Service Migration

Task Details

This task is to migrate the original analytics-services.coffee file to a new TypeScript file.

Component Purpose

The analytics service component collects and logs interactions the user has with the site. It allows a console output that shows what actions the user has taken in the environment. This component is linked in many places within the environment, meaning the task wasn’t just to migrate the old coffee script to a new type script, it also had to update each area it’s found in, and make it compatible with the new component.

Component Review of Original Task:

You can view the original task documentation here (thoth-tech/documentation#587)

First PR to Thoth-Tech:9.x

You can view this PR via this link (thoth-tech#292)

Type of Changes

  • Deleted: analytics-service.coffee
  • Added: analytics.service.ts
  • Modified: Other related files

Testing

Have tested in both Google and Safari.

Testing Summary
During the testing of this component, I went through the files that had the analytics service component in them, meaning that when that component was used, it would log analytics services in the console. With this, I was able to check that the analytics service was logging the events correctly.

Check List

  • My code follows the style guidelines of this project. (Angular 17 TypeScript)
  • I have performed a self-review of my code.
  • I have commented on my code in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings.
  • I have requested a review from two of my peers. Joe Iwasaki Maclean and Lovleen Kala and Lachlan Robinson.

@returnMarcco
Copy link

Hi @zara1504,
Can you please update the 'Migration Progress List' in README.MD.

Cheers

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