Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Major flows (e.g. sending cUSD, onboarding, etc.) analytics is easily accessible #4008

Closed
annakaz opened this issue Jun 4, 2020 · 1 comment
Assignees
Labels
end-user-experience End user experience bubble epic

Comments

@annakaz
Copy link
Contributor

annakaz commented Jun 4, 2020

Expected Behavior

Please describe the behavior you are expecting

Current Behavior

What is the current behavior?

@annakaz annakaz added epic end-user-experience End user experience bubble labels Jun 4, 2020
mergify bot pushed a commit that referenced this issue Jun 11, 2020
### Description
Created ability to add event properties for subEvent tracking and exposed those methods to the component used in the wallet.

Added analytics for the 2nd half of the Send flow (Ivan already did the first half), Secure Send, and the Request Payment flow.

### Other changes
Minor cleanup of dead code.

### Tested
No new tests added.

### Related issues
#3645 & #4008 

### Backwards compatibility
Yes
mergify bot pushed a commit that referenced this issue Jun 25, 2020
### Description

The new wallet KPIs require a few analytics we weren't already tracking. This PR adds balance and contact imports tracking, which is the final addition to be able to calculate the KPIs listed [here](https://docs.google.com/document/d/1G_XGpggW6cTBENcvs4v6XpaZzkTpST2D0kh31V9OVtU/edit). 

Note that transaction size analytics will be calculated using eksportisto data

### Tested

Not tested, no logic change- just adding analytics

### Related issues

- Related to epics #4008 and #4004 
- Fee tracking to help identify the extent of #4010 

### Backwards compatibility

Yes
mergify bot pushed a commit that referenced this issue Jul 1, 2020
### Description
Restructured and renamed the Valora wallet analytics. Changes include:
- Organized analytics events by category and introduced type checking for parameters
- Refactored all analytics parameters to be primitive data types to avoid pipeline or database errors
- Created a sessionId property and attach it and user address on each logged event
- Deleted unused or redundant events and parameters
- Consolidated all analytics into the `mobile` directory (deleted parent component in `react-components`)
- Removed subevent logging
- Renaming `CeloAnalytics` to `ValoraAnalytics`


To cover in next PR:
- Standardize event naming
- Further consolidate events
- Audit against [analytics doc](https://docs.google.com/document/d/1OGNJBijiWp7bWc2I0p8yyJ72EMCj9pNRT1VWxJJu5JY/edit?pli=1#heading=h.wip61atcgd2c)
- Introduce new events to fill gaps
- Update documentation

### Other changes
Fixed a couple of events that were broken.

### Tested
Yes

### Related issues
- Part of #4008

### Backwards compatibility
Yes
mergify bot pushed a commit that referenced this issue Jul 6, 2020
…4311)

### Description
Added events throughout the app and standardized naming and placement. Also added copyable sessionId to the drawer menu and including it in support emails to aid in debugging.

### Other changes
Using specific analytics naming to make all related events easily searchable. For example, search "SendEvents" to view all analytics events related to the Send flow.

### Tested
Yes

### Related issues
- Part of #4008

### Backwards compatibility
Yes
@annakaz
Copy link
Contributor Author

annakaz commented Jul 22, 2020

Closing this as ValoraAnalytics has what we need instrumented and can be viewed through dbt/metabase

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
end-user-experience End user experience bubble epic
Projects
None yet
Development

No branches or pull requests

2 participants