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

Display total number of actions across all websites #8550

Merged
merged 3 commits into from Aug 13, 2015
Merged

Display total number of actions across all websites #8550

merged 3 commits into from Aug 13, 2015

Conversation

mnapoli
Copy link
Contributor

@mnapoli mnapoli commented Aug 11, 2015

Fixes #8288

Show the total number of actions in the header:

capture d ecran 2015-08-12 a 10 31 41

I had to change the translation key TotalVisitsPageviewsRevenue to TotalVisitsPageviewsActionsRevenue, is it worth mentioning it in the changelog?

@mnapoli mnapoli added the Enhancement For new feature suggestions that enhance Matomo's capabilities or add a new report, new API etc. label Aug 11, 2015
@mnapoli mnapoli added this to the 2.15.0 milestone Aug 11, 2015
@tsteur
Copy link
Member

tsteur commented Aug 12, 2015

Was the issue only about adding total number of actions and later, in another issue, let people customize the dashboard so that they can add actions if needed? I'm not sure if I understood the issue right. Actions itself doesn't seem like a high value metric there (but in some cases it can be more value than pageviews)

@mnapoli
Copy link
Contributor Author

mnapoli commented Aug 12, 2015

I went with the quick fix, see #8288 (comment)

This is really high value for us, so targetting for 2.15.0 - we will need to find a solution, either a quick fix (display the number somewhere in the UI), or the more complicated - but terribly awesome - feature of letting users customise columns on All Websites dashboard: #2390

@tsteur
Copy link
Member

tsteur commented Aug 12, 2015

(display the number somewhere in the UI)

I think it's only meant the total number but not sure. @mattab ?

@mattab
Copy link
Member

mattab commented Aug 12, 2015

Maybe we don't really need the new column in the dashboard and we could only display the total number at the top?

@quba
Copy link
Contributor

quba commented Aug 12, 2015

Total number should suffice for now. But ability to customise would be really useful as well.

@mnapoli
Copy link
Contributor Author

mnapoli commented Aug 12, 2015

I've updated the PR (and the screenshot above) to remove the "Actions" column. I'll try to look at customizable columns, hopefully this is easy. In the meantime I think it shouldn't block this PR.

@tsteur
Copy link
Member

tsteur commented Aug 12, 2015

Re customizing columns: I'm not sure if it will be done in config or so but it would be nice if it was possible to configure it per site or my measurable type (website, mobile app, car, ...) one day. It might be not yet needed but in the future might be useful to think about it (eg how it could be displayed when different sites want to show different metrics etc). Use case: A B2B website has different relevant metrics than a B2C shop than a car than a mobile app

@tsteur
Copy link
Member

tsteur commented Aug 13, 2015

Can this one be merged?

@tsteur
Copy link
Member

tsteur commented Aug 13, 2015

There's no "needs review". Looks good otherwise

@mattab
Copy link
Member

mattab commented Aug 13, 2015

Very nice!

mattab pushed a commit that referenced this pull request Aug 13, 2015
Display total number of actions across all websites
@mattab mattab merged commit 539a619 into master Aug 13, 2015
@mnapoli mnapoli deleted the 8288 branch August 13, 2015 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement For new feature suggestions that enhance Matomo's capabilities or add a new report, new API etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants