Skip to content
Snippets Groups Projects
Commit d1fbab63 authored by Sameer Al-Sakran's avatar Sameer Al-Sakran
Browse files

Merge pull request #232 from metabase/saner-timeseries-requirements

loosen restrictions on timeseries history
parents 5b201cc2 ed070a96
Branches
Tags
No related merge requests found
......@@ -994,7 +994,7 @@ CardControllers.controller('CardDetail', [
}
//timeseries
if (data.cols.length > 1 && hasAscendingDate && hasScalar && result.row_count <= AXIS_CHART_MAX_ROWS) {
if (data.cols.length > 1 && hasAscendingDate && hasScalar) {
$scope.displayTypes.timeseries.available = true;
} else {
if (data.cols.length < 2) {
......@@ -1006,9 +1006,6 @@ CardControllers.controller('CardDetail', [
if (!hasAscendingDate) {
$scope.displayTypes.timeseries.notAvailableReasons.push("The data must contain at least one ascending date column");
}
if (result.row_count > AXIS_CHART_MAX_ROWS) {
$scope.displayTypes.timeseries.notAvailableReasons.push("The data must contain no more than " + AXIS_CHART_MAX_ROWS + " rows");
}
}
//pin map
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment