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

Monthly scheduled reports #10689

Closed
mgonera opened this issue Oct 5, 2016 · 7 comments
Closed

Monthly scheduled reports #10689

mgonera opened this issue Oct 5, 2016 · 7 comments
Assignees
Labels
answered For when a question was asked and we referred to forum or answered it.
Milestone

Comments

@mgonera
Copy link

mgonera commented Oct 5, 2016

Scheduled report fetches data using &period=range&date=last30 instead of fetching data from previous month. Can this be configured somehow or maybe desired behavior would be to "intelligently" choose previous actual month or week?

@tsteur
Copy link
Member

tsteur commented Oct 5, 2016

I don't think it can be configured

@tsteur
Copy link
Member

tsteur commented Oct 5, 2016

Adding label bug to this, haven't checked in the code but sounds buggy

@tsteur tsteur added the Bug For errors / faults / flaws / inconsistencies etc. label Oct 5, 2016
@mattab
Copy link
Member

mattab commented Oct 5, 2016

If that's the case it's definitely a bug, as the report should include last month report, not last 30 days. @mgonera how did you investigate this issue and found it was using last30 ?

@mgonera
Copy link
Author

mgonera commented Oct 6, 2016

Customer reached out with question on why there is no unique visitors number in monthly report.

After trying to recreate report definition and few other steps, I just checked what is the API request behind "Download" link of the report. It turned out that period=range&date=last30.

Obviously when I took the request that returns HTML report and changed period to month and date to specific date within desired month, all of the metrics, along with Unique Visitors number, were returned in the report.

@mgonera
Copy link
Author

mgonera commented Oct 6, 2016

I actually noticed that what is in the Report Download link, will rely on previously chosen UI context. And in this case range/last30 came from the fact, that these were default dashboard settings for particular user. Still I think this is bug, that report with "monthly" parameter can be downloaded for not-month period. Would you say we can expect improvement in this feature?

More importantly, I wonder what is actually being processed into the reports that are automatically sent by archiving process running for example on 1st day of the month. Because these did not contain the unique user metric originally. Could you maybe answer on that please?

@tsteur
Copy link
Member

tsteur commented Oct 6, 2016

I had a quick look in the code and there it should actually use the week, month, ... when sending the report. As for the UI the behaviour is expected but could be more clear. That's why there is a date selector above. @mattab I think we can close this one as it seems to work fine?

@mgonera
Copy link
Author

mgonera commented Oct 6, 2016

@tsteur ok, we can investigate it further on our side.

@tsteur tsteur closed this as completed Oct 6, 2016
@tsteur tsteur added answered For when a question was asked and we referred to forum or answered it. and removed Bug For errors / faults / flaws / inconsistencies etc. labels Oct 6, 2016
@tsteur tsteur added this to the 2.17.0 milestone Oct 6, 2016
@tsteur tsteur self-assigned this Oct 6, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
answered For when a question was asked and we referred to forum or answered it.
Projects
None yet
Development

No branches or pull requests

3 participants