Add comment about why the 'actions' property is being patched on

SteveSandersonMS · SteveSandersonMS · commit 0a0afed84b90 · 2016-08-18T13:57:49.000-07:00
diff --git a/samples/react/ReactGrid/ReactApp/components/PeopleGrid.jsx b/samples/react/ReactGrid/ReactApp/components/PeopleGrid.jsx
@@ -5,7 +5,9 @@ import{fakeData } from '../data/fakeData.js';
 import{columnMeta } from '../data/columnMeta.jsx';
 const resultsPerPage = 10;
 
-const fakeDataWithAction = fakeData.map(data => Object.assign(data,{actions: ''}));
+// Griddle requires each row to have a property matching each column, even if you're not displaying
+// any data from the row in that column
+fakeData.forEach(row =>{row.actions = ''});
 
 export class PeopleGrid extends React.Component{
 render(){
@@ -14,7 +16,7 @@ export class PeopleGrid extends React.Component{
 <div>
 <h1>People</h1>
 <div id="table-area">
- <Griddle results={fakeDataWithAction}
+ <Griddle results={fakeData}
 columns={columnMeta.map(x => x.columnName)}
 columnMetadata={columnMeta}
 resultsPerPage={resultsPerPage}

-Original file line number
+Diff line change
 import{columnMeta}from'../data/columnMeta.jsx';
 constresultsPerPage=10;
 -constfakeDataWithAction=fakeData.map(data=>Object.assign(data,{actions: ''}));
 +// Griddle requires each row to have a property matching each column, even if you're not displaying
 +// any data from the row in that column
 +fakeData.forEach(row=>{row.actions='';});
 exportclassPeopleGridextendsReact.Component{
 render(){
 <div>
 <h1>People</h1>
 <divid="table-area">
 -<Griddleresults={fakeDataWithAction}
 +<Griddleresults={fakeData}
 columns={columnMeta.map(x=>x.columnName)}
 columnMetadata={columnMeta}
 resultsPerPage={resultsPerPage}