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 message why no reports are shown if there are no conversions #9958

Merged
merged 1 commit into from Mar 29, 2016

Conversation

sgiehl
Copy link
Member

@sgiehl sgiehl commented Mar 22, 2016

fixes #9909

@sgiehl sgiehl added Help wanted Beginner friendly issues or issues where we'd highly appreciate community's help and involvement. not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org. Needs Review PRs that need a code review labels Mar 22, 2016
@sgiehl sgiehl added this to the 2.16.x (LTS) milestone Mar 22, 2016
@tsteur
Copy link
Member

tsteur commented Mar 29, 2016

FYI: This needs to be rebuilt in Piwik 3. Created #9972

@tsteur
Copy link
Member

tsteur commented Mar 29, 2016

It currently shows like this:

image

To be a bit more clear can we maybe write something like "Goal $goalName conversions by type of visit are not shown ..." and "Conversions overview by type of visit is not shown"

@tsteur
Copy link
Member

tsteur commented Mar 29, 2016

Maybe we could even still show the headline as usual and below write "there is no data for this report because there were no conversions ... "

@sgiehl
Copy link
Member Author

sgiehl commented Mar 29, 2016

Now the headline should still be visible when the message is shown.

@tsteur
Copy link
Member

tsteur commented Mar 29, 2016

👍

@tsteur tsteur merged commit 4cf7a64 into master Mar 29, 2016
@tsteur tsteur deleted the 9909 branch March 29, 2016 21:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Help wanted Beginner friendly issues or issues where we'd highly appreciate community's help and involvement. Needs Review PRs that need a code review not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants