Khan / perseus / 5425692780

69%

push

github

Remove virtualKeypadVersion from perseus anlaytics event (#585)

## Summary:

When I created the initial analytics event for driving out how analytics would look in perseus (`perseus:expression-evaluated`), I was hoping we'd be able to figure out which keypad the expression widget was using. But in most cases (today) that is decided by the host application. 

~So I'm going to remove it from the event and force the host application to fill that in before sending it on to CEDAR.~

EDIT: After talking with @handeyeco, we decided to make this key optional. We will be able to fill it in within Perseus sometimes, but not always.

Issue: LC-860

## Test plan:

`yarn test`

Author: jeremywiebe

Reviewers: jaredly, handeyeco, nedredmond, eamspoker

Required Reviewers:

Approved By: jaredly

Checks: ✅ finish_coverage, ✅ Publish npm snapshot (ubuntu-latest, 16.x), ✅ Extract i18n strings (ubuntu-latest, 16.x), ✅ Check builds for changes in size (ubuntu-latest, 16.x), ✅ Check for .changeset file (ubuntu-latest, 16.x), ✅ Jest Coverage (ubuntu-latest, 16.x), ✅ Cypress Coverage (ubuntu-latest, 16.x), ✅ Lint, Typecheck, and Test (ubuntu-latest, 16.x), ✅ gerald

Pull Request URL: https://github.com/Khan/perseus/pull/585

7536 of 16328 branches covered (46.15%)

Branch coverage included in aggregate %.

20102 of 44492 relevant lines covered (45.18%)

1003.57 hits per line