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

Stats: Summary Data missing for a single day in-app #10421

Closed
ghost opened this issue Nov 7, 2018 · 7 comments
Closed

Stats: Summary Data missing for a single day in-app #10421

ghost opened this issue Nov 7, 2018 · 7 comments

Comments

@ghost
Copy link

ghost commented Nov 7, 2018

Expected behavior

As a user, I should be able to see all of my stats for a given site. This data should be consistent across platforms.

Actual behavior

For a Brazilian user noted in p1541539436037700-slack-cellular-happiness, she does not see here stats on November 4. These stats are visible on the web.

Steps to reproduce the behavior

This is a user-reported issue. We believe the locale is set to pt-BR right. It's noteworthy that this was observed after a daylight savings time transition.

Tested on [device], iOS [version], WPiOS [version]

iPhone, v11.0 ("up to date"), WP v11.0 ("up to date")

@ghost ghost changed the title Stats: Stats: Summary Data missing for a single day in-app Nov 7, 2018
@ghost
Copy link
Author

ghost commented Nov 8, 2018

I asked internally in p1541622742023000-slack-portuguese-support, and another use is experiencing the same issue. In this case, however, the user's device language is set to English.

iPhone settings for second user with this issue.
10421a
10421b

@ghost
Copy link
Author

ghost commented Nov 8, 2018

I took some time to research the iOS implementation:

  • StatsViewController uses uses -[BlogService timeZoneForBlog:]
  • The value returned (e.g., GMT or America/Los Angeles is set on WPStatsViewController and injected into WPStatsService).
  • WPStatsService is injected into InsightsTableViewController, where the service is invoked.
  • The time zone value appears to be injected properly, but is never used in practice within WPStatsServiceRemote.
  • The uses of time zone appear to be either [NSTimeZone localTimeZone] or [NSTimeZone timeZoneForSecondsFromGMT:0].
  • This likely skews the parameters for before, after and date passed in requests.

cc: @ScoutHarris - for consideration while working on the Full Stats Refresh project.

@ghost
Copy link
Author

ghost commented Nov 8, 2018

Additional impact:
10421c

@ScoutHarris
Copy link
Contributor

Thanks for doing the research @stevebaranski ! We'll keep this in mind.

cc @jklausa

@jamiepalatnik
Copy link

jamiepalatnik commented Nov 8, 2018

Reported in #1545121-zen

The user sent these screenshots.
date time
language region

@ScoutHarris
Copy link
Contributor

@rachelmcr - can you remove this from the Groundskeeping project? I'm not sure why I can't, but this will be addressed with the Stats Refresh project. Thank you!!

@ScoutHarris
Copy link
Contributor

Fixed with Stats Refresh.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants