webex / webex-js-sdk

JavaScript SDK for Webex
https://webex.github.io/webex-js-sdk/
Other
174 stars 343 forks source link

fix: circular dependency issues #3628

Closed mkesavan13 closed 3 months ago

mkesavan13 commented 4 months ago

PARTIALLY COMPLETES SPARK-515520

This pull request addresses

Solves the following circular dependencies:

by making the following changes

For call-diagnostic-metrics.ts <-> new-metrics.ts,

getOSNameInternal was imported from the whole plugin instead of the internal file. Changed that to import from the internal file

For internal-plugin-device <-> internal-plugin-metrics

Here, the metrics require some device info to be sent to amplitude for which it imports the internal-plugin-device and as part of this PR, we made changes to ensure internal-plugin-device sets the whole device plugin to the call diagnostics metrics on successful registration of a device

Change Type

The following scenarios where tested

This is when I entered my PMR and logged in: Screenshot 2024-06-10 at 6 03 06 AM

This is when I just loaded localhost:4000/meeting that redirected automatically to localhost:4000/join-a-meeting: Screenshot 2024-06-10 at 6 04 34 AM

I certified that


Make sure to have followed the contributing guidelines before submitting.