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

Events reports: option to view all columns: min, max and avg values #6761

Closed
mattab opened this issue Dec 1, 2014 · 7 comments
Closed

Events reports: option to view all columns: min, max and avg values #6761

mattab opened this issue Dec 1, 2014 · 7 comments
Assignees
Milestone

Comments

@mattab
Copy link
Member

@mattab mattab commented Dec 1, 2014

The goal of this issue is to improve the Events reports to add a new option in the Cog icon, to Show value metrics. When clicked, the following columns will be displayed on top of existing ones:

  • Average value
  • Min value
  • Max value
  • Event with a value (on hover, show % relative value)
@tsteur

This comment has been minimized.

Copy link
Member

@tsteur tsteur commented Oct 28, 2015

This would be real useful for us see #9115 . Without this the events are meaningless for us and I presume for many other users as well. It should be maybe even show all columns by default or by smart and detect when there is an avg value of 1 (and min value = 1 and max value = 1) and not show the other metrics in this case.

@mattab

This comment has been minimized.

Copy link
Member Author

@mattab mattab commented Oct 29, 2015

indeed, now that we use it we realise how much crucial this feature is... Moved into 2.15.1

@hpvd

This comment has been minimized.

Copy link

@hpvd hpvd commented Oct 30, 2015

now that we use it we realise how much crucial this feature is... Moved into 2.15.1

=> wow that's GREAT to here - we hoped you would notice this :-)

If you want to make event statistics more usable and meaningful in general within 2.15.1,
these topics may be also interesting to be moved to this milestone:

  1. "Visitor log: Show time and nr of action/events for each site directly" #8731
  2. "Visitor engagement: Show list of pages with most actions/events" within #8914
@tsteur

This comment has been minimized.

Copy link
Member

@tsteur tsteur commented Dec 15, 2015

For consistency, I wonder if it wasn't more appropriate to use visualization tableAllColumns for this instead of the cog icon? This would probably also avoid adding several new features to core just for this.

@tsteur tsteur self-assigned this Dec 15, 2015
tsteur added a commit that referenced this issue Dec 15, 2015
@tsteur

This comment has been minimized.

Copy link
Member

@tsteur tsteur commented Dec 15, 2015

I'm adding this feature via tableAllColumns here: https://github.com/piwik/piwik/compare/6761?expand=1

It looks like this:

image

Current problem: It seems to not remember the visualization when clicking on any of the ReportByDimension left menu links. Need to look into it.

Also I noticed there are two more columns available and I'm not sure if we should show them: "Visits" and sometimes "Unique Visitors" depending of period and Piwik setting. Does it make sense to show these columns there? Also should we show nb_events_with_value?

@mattab

This comment has been minimized.

Copy link
Member Author

@mattab mattab commented Dec 21, 2015

Feedback

  • Nice idea to re-use the table all columns icon for this
  • the "Footer expanded/collapsed" state is not restored/persisted, it would be nice if it was (not in the scope of this issue so optional)

Also I noticed there are two more columns available and I'm not sure if we should show them: "Visits" and sometimes "Unique Visitors" depending of period and Piwik setting. Does it make sense to show these columns there?

It does make sense to show them 👍

Also should we show nb_events_with_value?

👍

@tsteur

This comment has been minimized.

Copy link
Member

@tsteur tsteur commented Dec 21, 2015

Re state restore see #9398

tsteur added a commit that referenced this issue Dec 21, 2015
@mattab mattab closed this in #9399 Dec 22, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.