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
Show 'new' metrics on visitor engagement report/sparklines #15128
Conversation
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.
@katebutler left a few comments.
# Conflicts: # plugins/Ecommerce/tests/System/expected/test_ecommerceOrderWithItems_schedrep_in_pdf_tables_only__ScheduledReports.generateReport_week.original.pdf # tests/PHPUnit/System/expected/test_DataComparisonTest_multipleMultiPeriods__VisitFrequency.get_day.xml # tests/PHPUnit/System/expected/test_DataComparisonTest_multipleSites_multipleCompare__VisitFrequency.get_month.xml # tests/PHPUnit/System/expected/test_TwoVisitors_twoWebsites_differentDays_schedrep_in_pdf_tables_only__ScheduledReports.generateReport_month.original.pdf
# Conflicts: # tests/PHPUnit/System/expected/test_BackwardsCompatibility1XTest__VisitFrequency.get_range.xml # tests/PHPUnit/System/expected/test_BackwardsCompatibility1XTest__VisitFrequency.get_year.xml # tests/PHPUnit/System/expected/test_BackwardsCompatibility1XTest_multipleOldNew__VisitFrequency.get_month.xml
… new or returning ones (unable to trigger this as the evolution graph doesn't call the method at all in this case); ensure that only the columns requested will be returned
…tests to check that correct columns are returned
@katebutler looking at the tests there seem to be heaps of them failing? |
@katebutler can you update the ecommerce tests as discussed last Thursday or Friday? Then we can merge |
$columns = Piwik::getArrayFromApiParameter($columns); | ||
|
||
/** @var \Piwik\DataTable\DataTableInterface $resultSet */ | ||
$resultSet = new DataTable(); |
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.
FYI, this needs to be new Simple()
otherwise the XML output will have an extra <row>
element.
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.
cheers @diosmosis
* Switch VisitFrequency to use Simple datatable, restore constant in Goals API for BC * Update system test files
* Switch VisitFrequency to use Simple datatable, restore constant in Goals API for BC * Update system test files
Fixes #539