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

Clicking "Change period" feature can result in the evolution graph showing not needed information #18631

Open
mattab opened this issue Jan 17, 2022 · 2 comments
Labels
Bug For errors / faults / flaws / inconsistencies etc.

Comments

@mattab
Copy link
Member

mattab commented Jan 17, 2022

Expected Behavior

Evolution graphs across Matomo should show the correct data and not show wrong information.
(And the last graph section should be dotted if the selected period includes the current unfinished period)

Current Behavior

Graph shows 0 values for periods in the future (the graph is too much in the future)

Possible Solution

Steps to Reproduce (for Bugs)

  1. go to https://demo.matomo.cloud/index.php?module=CoreHome&action=index&idSite=1&period=day&date=yesterday#?idSite=1&period=year&date=2022-01-16&segment=&category=Goals_Goals&subcategory=General_Overview (which is in Goals > Overview with the year selected)
  2. Click "Change period"
  3. Select "Month" period (instead of the current "Year")
  4. the graph shows 12 months in the future with zero values:

Screenshot from 2022-01-17 14-10-10

Note: a similar problem occur if you're looking at a "Month" and change period to a Week. (it then shows a few weeks in the future)

Context

just using a core UI feature to get some insights

@mattab mattab added the Bug For errors / faults / flaws / inconsistencies etc. label Jan 17, 2022
@sgiehl
Copy link
Member

sgiehl commented Jan 17, 2022

@mattab What exactly would you expect? Shall we hide all periods that are in the future, or shall we still show them, but everything in the future would be dotted?

@mattab
Copy link
Member Author

mattab commented Jan 18, 2022

@sgiehl 👍 hide all periods in the future (because they are always zero-values)
(and when the current unfinished period is the selected one, then it's also shown as dotted in the graph)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Projects
None yet
Development

No branches or pull requests

3 participants