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
convert sparkline api to one request - fixes archive #18493
Conversation
update lock
update sparkline
update title link
add comments
update loader
update template
update to visualise
update controller
revert lock
update matrix
update config title
remove template
remove mutilple
us getMetrics
update tests
Co-authored-by: Ben Burgess <88810029+bx80@users.noreply.github.com>
Co-authored-by: Ben Burgess <88810029+bx80@users.noreply.github.com>
# Conflicts: # plugins/CoreHome/vue/dist/CoreHome.umd.min.js
update umd js
update goal remove revenue sparkline
remove hide extra columns
update graphic
this is the follow-up on the UI failed fixes. Should pass the test now. |
@peterhashair The only remaining failing UI screenshot |
update remove goal in the settings
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.
UI tests are now all passing.
I noticed that because the GetMetrics
report is a now a descendant of the Get
report that the view is configured slightly differently with overview Goal Names shown as Goal 'Something'
instead of Goal Something
. This would seem to be more consistent, what do you think @sgiehl ?
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.
Generally seems to work now. Left a couple of comments for code improvements
Co-authored-by: Stefan Giehl <stefan@matomo.org>
Co-authored-by: Stefan Giehl <stefan@matomo.org>
remove check actions and remove foreach loop on goals
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.
Had another idea for further code improvement. Otherwise this should be good to merge, if tests are passing.
Co-authored-by: Stefan Giehl <stefan@matomo.org>
This reverts commit 36f83c1.
Description:
Description:
Fixes: #18088.
Follow the fix on UI tests
Review