chore(feature): knowledge hub Telemetry Setup

[aakankshagupta1994]

Issue

#35
#36

Description

TBI - Provide telemetry data for the tutorials and tutorial tags opened
TBI - Provide telemetry data for the blogs and tags opened

FILL IN DETAILS HERE

Checklist for Pull Requests

• Supplied as many details as possible on this change
• Included the link to the associated issue in the Issue section above
• The code conforms to the general development guidelines.
• The code has unit tests where applicable and is easily unit-testable
• This branch is appropriately named for the associated issue

[changeset-bot]

⚠️ No Changeset found

Latest commit: f373855

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[kranthie-sap]

Hi @aakankshagupta1994, please see my review comments. Also, please update the ide-extension telemetry unit tests to use the right telemetry events send from the webapp. Also, please add unit tests in the webapp to simulate clicks and check if the action is triggered.

[kranthie-sap]

@aakankshagupta1994: I just noticed that sonar reported a bug in the PR - https://sonarcloud.io/project/issues?id=SAP_knowledge-hub-extension&pullRequest=68&resolved=false&types=BUG

[aakankshagupta1994]

@kranthie-sap The requested changes are added. Can you please verify?

[Klaus-Keller]

@kranthie-sap @aakankshagupta1994
the two tracked telemetry events in this pull request are for links that are clicked and need a new redux action, so far so good. However, it is often the case that an existing redux action can be used as trigger to track telemetry data. In this case, to avoid polluting application logic with telemetry events an alternative is to add a telemetry middleware and filter/map redux actions that should be tracked.
We can discuss this along with this PR or postpone the discussion until more events are added, whatever you prefer.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

82.0% Coverage
0.0% Duplication

[Klaus-Keller]

Hi @aakankshagupta1994,

Thanks for addressing all my comments. There are no open issues that I can see, from my side this is approved.