Skip to content

Commit

Permalink
include out of scope items when exporting and ignore scope is on
Browse files Browse the repository at this point in the history
  • Loading branch information
tyrjo committed Nov 7, 2018
1 parent 1d1af04 commit c91703c
Show file tree
Hide file tree
Showing 5 changed files with 34 additions and 18 deletions.
26 changes: 13 additions & 13 deletions CustomChartApp.js
Original file line number Diff line number Diff line change
Expand Up @@ -233,19 +233,19 @@ Ext.define('CustomChartApp', {
model = this.models[0],
config = {
xtype: chartType,
enableStacking: !!stackField,
chartColors: [
"#FF8200", // $orange
"#F6A900", // $gold
"#FAD200", // $yellow
"#8DC63F", // $lime
"#1E7C00", // $green_dk
"#337EC6", // $blue_link
"#005EB8", // $blue
"#7832A5", // $purple,
"#DA1884", // $pink,
"#C0C0C0" // $grey4
],
enableStacking: !!stackField,
chartColors: [
"#FF8200", // $orange
"#F6A900", // $gold
"#FAD200", // $yellow
"#8DC63F", // $lime
"#1E7C00", // $green_dk
"#337EC6", // $blue_link
"#005EB8", // $blue
"#7832A5", // $purple,
"#DA1884", // $pink,
"#C0C0C0" // $grey4
],
storeConfig: {
context: this.getContext().getDataContext(),
//TODO: can we do summary fetch here and not limit infinity?
Expand Down
15 changes: 15 additions & 0 deletions Overrides.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
Ext.override(Rally.ui.gridboard.Export, {
// Override to also look at the cmp initial config storeConfig value
_getScopeParams: function(cmp) {
var context = (cmp.store && cmp.store.context) ||
(cmp.initialConfig && cmp.initialConfig.storeConfig && cmp.initialConfig.storeConfig.context) ||
cmp.getContext().getDataContext();

return {
workspace: context.workspace,
project: context.project,
projectScopeDown: context.projectScopeDown,
projectScopeUp: context.projectScopeUp
};
}
});
2 changes: 1 addition & 1 deletion column/Chart.js
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ Ext.define('ColumnChart', {

this.chartConfig.plotOptions.column.showInLegend = this.enableStacking;
this.chartConfig.plotOptions.column.colorByPoint = !this.enableStacking;

if (!this.enableStacking) {
this.chartConfig.tooltip = {
headerFormat: '',
Expand Down
1 change: 1 addition & 0 deletions config.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
"server": "https://rally1.rallydev.com",
"sdk": "2.1",
"javascript": [
"Overrides.js",
"node_modules/@agile-central-technical-services/utils-ancestor-pi-app-filter/index.js",
"node_modules/@agile-central-technical-services/utils-ancestor-pi-inline-filter/index.js",
"ChartUtils.js",
Expand Down
8 changes: 4 additions & 4 deletions deploy/Ugly.txt

Large diffs are not rendered by default.

0 comments on commit c91703c

Please sign in to comment.