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

Fix AutorecordInstalls / Updated App bug #32

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dataxpress
Copy link

Current behavior: Updated App is sent for every app launch in which the user's app version is not equal to the first installed version
E.G.

User launches 1.0 -> Installed App     // correct behavior
User launches 1.0 -> <No event sent>   // correct behavior
User launches 1.1 -> Updated App       // correct behavior
User launches 1.1 -> Updated App       // *incorrect* behavior - expected no event sent

Expected behavior: Updated App is only sent on the first launch after the app is updated.

This update fixes the issue by recording the latest app version when an update is detected.

Current behavior: `Updated App` is sent for every app launch in which the user's app version is not equal to the first installed version
E.G.
User launches 1.0 -> Installed App   // correct behavior
User launches 1.0 -> <No event sent>  // correct behavior
User launches 1.1 -> Updated App     // correct behavior
User launches 1.1 -> Updated App   // *incorrect* behavior - expected no event sent

This update fixes the issue by recording the latest app version when an update is detected.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant