Khan / perseus / 5931935552

69%

main: 69%

Build Type

Pull #606

github

Committed by web-flow

Commit Message

Derive virtual keypad version (#684)

## Summary:

As I was integrating Perseus analytics in webapp, it became really difficult to work out how to provide the virtualKeypadVersion consistently when Perseus wasn't always providing it. After some thought, I figured out that we can pretty-reliably derive the keypad version. Given that we are actively moving to coalesce on a single keypad, I hope this is an acceptable approach for the short term. When we get to one keypad, we can delete this `virtualKeypadVersion` from the analytics anyways. 

Issue: LC-950

## Test plan:

✅ `yarn typecheck`
✅ `yarn test`

Author: jeremywiebe

Reviewers: nedredmond, SonicScrewdriver, handeyeco

Required Reviewers:

Approved By: nedredmond

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

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

Pull Request Pull Request #606: Feature: Analytics

Run Details

7625 of 16588 branches covered (45.97%)

Branch coverage included in aggregate %.

235 of 378 new or added lines in 19 files covered. (62.17%)

6 existing lines in 1 file now uncovered.

71380 of 97404 relevant lines covered (73.28%)

710.29 hits per line