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

Tracking persistence layer refactor #119

Merged
merged 5 commits into from
Oct 21, 2021
Merged

Tracking persistence layer refactor #119

merged 5 commits into from
Oct 21, 2021

Conversation

zihejia
Copy link
Contributor

@zihejia zihejia commented Oct 20, 2021

Replace the file persistence layer with PlayerPref and remove the multi-threading to reduce memory and CPU consumption. This change is to address #108 #118 #116 #110 #90 #106 #112 #69 and will support any IL2CPP build and WebGL.
Other changes:

  • Better logging for network request
  • Add ClearSuperProperties
  • Add TrackCharge(double amount, Value properties )

@zihejia zihejia added the bug label Oct 20, 2021
@zihejia zihejia requested a review from jaredmixpanel October 20, 2021 19:03
Copy link
Contributor

@jaredmixpanel jaredmixpanel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm... love seeing all that deleted code!

@zihejia zihejia added enhancement and removed bug labels Oct 21, 2021
@zihejia zihejia merged commit 22d18bc into master Oct 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants