fix #272: add event telemetry for incompat addons
This commit is contained in:
parent
ad3c227150
commit
4db04e1df5
2 changed files with 16 additions and 1 deletions
|
@ -169,6 +169,15 @@ of a `testpilottest` telemetry ping for each scenario.
|
|||
}
|
||||
```
|
||||
|
||||
* When a user encounters the disabled "move" feature because of incompatible add-ons
|
||||
|
||||
```js
|
||||
{
|
||||
"uuid": <uuid>,
|
||||
"event": "incompatible-addons-detected"
|
||||
}
|
||||
```
|
||||
|
||||
### A Redshift schema for the payload:
|
||||
|
||||
```lua
|
||||
|
|
8
index.js
8
index.js
|
@ -500,7 +500,13 @@ const ContainerService = {
|
|||
return new Promise(resolve => {
|
||||
AddonManager.getAddonsByIDs(INCOMPATIBLE_ADDON_IDS, (addons) => {
|
||||
addons = addons.filter((a) => a && a.isActive);
|
||||
resolve(addons.length !== 0);
|
||||
const incompatibleAddons = addons.length !== 0;
|
||||
if (incompatibleAddons) {
|
||||
this.sendTelemetryPayload({
|
||||
"event": "incompatible-addons-detected"
|
||||
});
|
||||
}
|
||||
resolve(incompatibleAddons);
|
||||
});
|
||||
});
|
||||
},
|
||||
|
|
Loading…
Add table
Reference in a new issue