-
Notifications
You must be signed in to change notification settings - Fork 9
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: last used format for device #287
Conversation
Sample app builds 📱Below you will find the list of the latest versions of the sample apps. It's recommended to always download the latest builds of the sample apps to accurately test the pull request.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #287 +/- ##
============================================
+ Coverage 54.35% 54.43% +0.07%
- Complexity 278 282 +4
============================================
Files 109 109
Lines 2535 2535
Branches 355 355
============================================
+ Hits 1378 1380 +2
Misses 1033 1033
+ Partials 124 122 -2 ☔ View full report in Codecov by Sentry. |
Build available to test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍🏻
## [3.9.1](3.9.0...3.9.1) (2024-03-21) ### Bug Fixes * last used format for device ([#287](#287)) ([726667b](726667b))
closes: https://linear.app/customerio/issue/MBL-170/incorrect-last-used-value-being-sent-from-android-sdk
Device payload was using
lastUsed
instead oflast_used
while adding a device/device attributes which BE doesn't accept.