feat: c-a views components events migration to segment

[tommasini]

Description

C-A views component events migration to segment

Related issues

Fixes:

Manual testing steps

1. Go to this page...

Screenshots/Recordings

Before

After

Pre-merge author checklist

• I’ve followed MetaMask Coding Standards.
• I've clearly explained what problem this PR is solving and how it is solved.
• I've linked related issues
• I've included manual testing steps
• I've included screenshots/recordings if applicable
• I’ve included tests if applicable
• I’ve documented my code using JSDoc format if applicable
• I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.
• I’ve properly set the pull request status:
  • In case it's not yet "ready for review", I've set it to "draft".
  • In case it's "ready for review", I've changed it from "draft" to "non-draft".

Pre-merge reviewer checklist

• I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
• I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

[github-actions]

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

[NicolasMassart]

Some issues to fix:

• trackDappVisitedEvent should not be updated here (and it should be moved instead of added)
• no need for empty properties object in trackEvent calls
• missing one InteractionManager removal

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 18.64407% with 48 lines in your changes are missing coverage. Please review.

Project coverage is 41.47%. Comparing base (baed762) to head (a92ef18).

Files	Patch %	Lines
app/components/Views/BrowserTab/index.js	0.00%	13 Missing ⚠️
...ts/Views/AccountPermissions/AccountPermissions.tsx	0.00%	7 Missing ⚠️
...components/Views/AccountConnect/AccountConnect.tsx	0.00%	6 Missing ⚠️
...ents/Views/AddAccountActions/AddAccountActions.tsx	0.00%	4 Missing ⚠️
...ountPermissionsRevoke/AccountPermissionsRevoke.tsx	0.00%	3 Missing ⚠️
app/components/Views/Browser/index.js	0.00%	3 Missing ⚠️
...rmissionsConnected/AccountPermissionsConnected.tsx	0.00%	2 Missing ⚠️
...mponents/Views/AccountSelector/AccountSelector.tsx	0.00%	2 Missing ⚠️
app/components/Views/ActivityView/index.js	0.00%	2 Missing ⚠️
app/components/Views/AssetDetails/index.tsx	0.00%	2 Missing ⚠️
... and 4 more

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #8711   +/-   ##
=======================================
  Coverage   41.46%   41.47%           
=======================================
  Files        1263     1264    +1     
  Lines       30610    30622   +12     
  Branches     3008     3008           
=======================================
+ Hits        12693    12700    +7     
- Misses      17164    17169    +5     
  Partials      753      753           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[NicolasMassart]

Looks good to me.

[NicolasMassart]

Looks good to me.

[sonarcloud]

Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
18.6% Coverage on New Code
18.2% Duplication on New Code

See analysis details on SonarCloud