Skip to content
This repository was archived by the owner on Dec 15, 2023. It is now read-only.

Updated telemetry format #110

Merged
merged 2 commits into from
Oct 15, 2019
Merged

Updated telemetry format #110

merged 2 commits into from
Oct 15, 2019

Conversation

stevehobbsdev
Copy link
Contributor

Changes

This PR updates the format of the telemetry object to current requirements, by sending the following object:

var telemetry = {
  name: 'auth0-cordova',
  version: version,
  env: {
    'auth0.js': '9.4.2',
    os: getOS()   // 'ios' or 'android'
  }
};

Testing

  • This change adds test coverage
  • This change has been tested on the latest compatible version of Cordova or why not

Checklist

@stevehobbsdev stevehobbsdev requested a review from a team October 14, 2019 13:01
@stevehobbsdev stevehobbsdev merged commit 7a874a3 into master Oct 15, 2019
@stevehobbsdev stevehobbsdev deleted the feature/telemetry branch October 15, 2019 16:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants