Detect for timezone changes and update the user #1542
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
One Line Summary
Send up timezone ID as a one-time migration, send up timezone ID when fetching the legacy player, and start detecting timezone changes and sending it up when it changes.
Details
session_count
update. This update is driven by reading thetimezone_id
from cache, but before this PR, timezone was never cached, so it will return null, driving this one-time update.Motivation
Reports by customers about missing timezones on very old players after migrating to v5.
Also requests by customers to detect timezone changes.
Scope
Sending timezone
Testing
Unit testing
None added
Manual testing
iPhone 13 on iOS 18.1
Scenario tested - player model upgrade:
Scenario tested - user model upgrade:
main
Scenario tested - new install:
Affected code checklist
Checklist
Overview
Testing
Final pass
This change is