Skip to content

Conversation

@manu-d
Copy link
Contributor

@manu-d manu-d commented Mar 12, 2018

No description provided.

@manu-d manu-d requested a review from xaun March 12, 2018 17:20
@manu-d manu-d changed the base branch from master to lmi-insights March 12, 2018 17:21
Copy link
Contributor

@xaun xaun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey looks good, just need to fix the data-not-found directive


$scope.chart = new HighchartsFactory($scope.chartId(), w.content.chart, options)

$scope.chart.formatters = ->
Copy link
Contributor

@xaun xaun Mar 19, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To save some lines, you can assign the result to a variable, and then mutate the object and return that.

defaultFormatters = $scope.chart.formatters()
$scope.chart.formatters = -> 
   defaultFormatters.xAxis = { new: 'opts' }
   ...
   defaultFormatters

</div>

<!-- No data found -->
<div ng-if="(isDataFound==false)" common-data-not-found on-display-alerts="onDisplayAlerts()" endpoint="::widget.category" width="::widget.width" />
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the old implementation of the data-not-found directive. I know the generator created it, but its out of date. I updated this in the generator in v1.6.9 but the lmi feature branch is behind.

The data-not-found is now a modal, so no need to show/hide content on $scope.isDataFound. Replace this directive with: <div ng-show="widget.demoData" common-data-not-found />.

Take a look at cash projection template for example, test with a bolt widget for org with no bolt data.

@xaun
Copy link
Contributor

xaun commented Mar 19, 2018

Also @manu-d, the JIRA says to load a generic Highcharts template?

@cesar-tonnoir cesar-tonnoir added this to the v1.8.0 milestone Apr 11, 2018
@cesar-tonnoir cesar-tonnoir modified the milestones: v1.8.0, v1.9.0 May 24, 2018
@agranado2k agranado2k modified the milestones: v1.9.0, LMI Jun 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants