Skip to content
This repository has been archived by the owner on Feb 1, 2024. It is now read-only.

[8](13) follow-on tasks for Kelp CLI data tracking #516

Closed
8 tasks done
nikhilsaraf opened this issue Sep 29, 2020 · 0 comments · Fixed by #536
Closed
8 tasks done

[8](13) follow-on tasks for Kelp CLI data tracking #516

nikhilsaraf opened this issue Sep 29, 2020 · 0 comments · Fixed by #536
Assignees
@debnil
Labels
feature request New feature or request
Milestone
v1.10.0

Comments

@nikhilsaraf
Copy link
Contributor

nikhilsaraf commented Sep 29, 2020 •
edited
Loading

Desired Behavior

I want to persist state of user preferences and userId in regards to data tracking

Impact

The desired behavior will allow me to be consistent with user prefs

Features

Additional context

related to #478
@nikhilsaraf nikhilsaraf added the feature request New feature or request label Sep 29, 2020
@nikhilsaraf nikhilsaraf added this to the v1.10.0 milestone Sep 29, 2020
This was referenced Sep 29, 2020
Merged
Closed
@nikhilsaraf nikhilsaraf changed the title [ad-hoc] use kelp prefs file during data tracking [ad-hoc] follow-on tasks for Kelp CLI data tracking Oct 9, 2020
@nikhilsaraf nikhilsaraf changed the title [ad-hoc] follow-on tasks for Kelp CLI data tracking [ad-hoc][3] follow-on tasks for Kelp CLI data tracking Oct 9, 2020
@nikhilsaraf nikhilsaraf changed the title [ad-hoc][3] follow-on tasks for Kelp CLI data tracking [ad-hoc][5] follow-on tasks for Kelp CLI data tracking Oct 9, 2020
@nikhilsaraf nikhilsaraf changed the title [ad-hoc][5] follow-on tasks for Kelp CLI data tracking [8] follow-on tasks for Kelp CLI data tracking Oct 13, 2020
This was referenced Oct 13, 2020
Merged
Merged
Merged
debnil added a commit that referenced this issue Oct 13, 2020
@debnil
Add isTestnet to metric common props (part of #516)
98b31b2 
PR #530, issue #516 

* Initial commit - b36caac

* Add SDEX check - f6c27cc
This was referenced Oct 14, 2020
Merged
Merged
debnil added a commit that referenced this issue Oct 14, 2020
@debnil
Add device ID to metric event props (part of #516) (#531)
474ccab 
* Initial commit - 2aad8e6

* Add glide files - 5efbdca

* Merge - e74a1c4
nikhilsaraf pushed a commit that referenced this issue Oct 14, 2020
@debnil
Send update event execution duration with metrics (part of #516) (#533)
f728875 
* Initial commit

* Rename as millisForUpdate

* Redefine millisForUpdate

* Address review - nikhilsaraf
nikhilsaraf pushed a commit that referenced this issue Oct 14, 2020
@debnil
Use hashed trading account for metrics user ID (part of #516) (#534)
730c1da 
* 1 - factor out hash function

* 2 - replace user ID with hash

* Factor out user ID function

* Use exchange API key

* Address review of function
nikhilsaraf pushed a commit that referenced this issue Oct 14, 2020
@debnil
Disable metrics tracking with no headers (part of #516) (#532)
411b795 
* Initial commit

* Fix build issue

* Remove testnet on metrics tracker.
nikhilsaraf pushed a commit that referenced this issue Oct 15, 2020
@debnil
Add official amplitude metric properties (part of #516) (#535)
657abc9
@debnil debnil mentioned this issue Oct 15, 2020
Merged
nikhilsaraf pushed a commit that referenced this issue Oct 17, 2020
@debnil
Send update metric at certain intervals (closes #516) (#536)
c341d64 
* Initial commit

* Address review - inject start time, add border tests

* Rename startTime, fix boundary condition.

* Add lastMetricUpdateTime to struct.

* Fix delete start time.

* Fix exporting.

* Remove debug.

* Change update time pattern.
@debnil debnil changed the title [8] follow-on tasks for Kelp CLI data tracking [8](13) follow-on tasks for Kelp CLI data tracking Oct 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@debnil debnil

Labels
feature request New feature or request
Projects
None yet
Milestone
v1.10.0
Development

Successfully merging a pull request may close this issue.

Send update metric at certain intervals (closes #516) debnil/kelp
2 participants
@nikhilsaraf @debnil