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

Hide duplicate actions in visitor log by default #12160

Merged
merged 7 commits into from Dec 20, 2017
Merged

Conversation

sgiehl
Copy link
Member

@sgiehl sgiehl commented Oct 6, 2017

...and extend them on click

fixes #9586

@sgiehl sgiehl added Enhancement For new feature suggestions that enhance Matomo's capabilities or add a new report, new API etc. Needs Review PRs that need a code review labels Oct 6, 2017
@sgiehl sgiehl force-pushed the actioncollapse branch 15 times, most recently from 1a92457 to 78eda2b Compare October 13, 2017 16:54
@sgiehl
Copy link
Member Author

sgiehl commented Oct 14, 2017

UI test failures are unrelated to this PR

@mattab mattab added this to the 3.2.1 milestone Oct 16, 2017
@mattab
Copy link
Member

mattab commented Nov 19, 2017

Could you add a UI test that clicks on the "Expand group" and check the actions expand as expected? Ideally the UI test would take a screenshot of only the visit including the expanded actions so it's clear what the UI test does.

@mattab
Copy link
Member

mattab commented Nov 19, 2017

  • @sgiehl let's also make this behavior consistent in the Visitor Profile

@mattab mattab modified the milestones: 3.2.1, 3.3.0, 3.2.2 Nov 19, 2017
@sgiehl
Copy link
Member Author

sgiehl commented Nov 21, 2017

I've update the PR to have consistent behavior on profile and log

@mattab
Copy link
Member

mattab commented Dec 14, 2017

@sgiehl Looks good 👍
Could you resolve the conflict, and merge this?

@sgiehl
Copy link
Member Author

sgiehl commented Dec 15, 2017

conflict resolved. all related tests are passing again...

@sgiehl sgiehl merged commit 786c3da into 3.x-dev Dec 20, 2017
@sgiehl sgiehl deleted the actioncollapse branch December 20, 2017 15:43
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. Needs Review PRs that need a code review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

visitor log: grouping hides information
2 participants