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
Improve / fix handling of best converting page on goal overview #19972
Conversation
update screenshots
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Working as expected, sorry pushed some UI screenshots that need an update, I believe one is from my PR that already merged.
- Functional review done
- Potential edge cases thought about (behavior of the code with strange input, with strange internal state or possible interactions with other Matomo subsystems)
- Usability review done (is anything maybe unclear or think about anything that would cause people to reach out to support)
- Security review done
- Wording review done
- Code review done
- Tests were added if useful/possible
- Reviewed for breaking changes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've done some functional testing, the tooltip works and broken URL is removed. 👍
The original issue mentions that a directory shouldn't be shown as the top entry URL. When testing locally my top entry page is shown as 'blog' which is a directory. The top dimensions request might need to be filtered to exclude directories?
@bx80 That should be fixed now. I've changed the request for entry pages and added |
update screenshot
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update some failed screenshots, looks good to me.
Description:
fixes #19512
fixes #20003
Review