-
Notifications
You must be signed in to change notification settings - Fork 7
feat(oui-datagrid): add datagrid customization #196
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.
Consoles to remove.
Apart that, it's ok for me
} | ||
|
||
this.columns = builtColumns.columns.filter(column => !column.hidden); | ||
console.log(this.columnsParameters); |
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.
There is a console here to remove
return column; | ||
}); | ||
|
||
console.log(this.availableColumns); |
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.
Here too
const customColumn = find(this.columnsParameters, { | ||
name: column.name | ||
}); | ||
console.log(customColumn, column.name); |
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.
And a last one here :)
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.
Oops
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.
BTW, you can easily make build fail with that on CI if you want.
4cfb2af
to
6783b0d
Compare
SCDC-744
PR Doc: ovh-ux/ovh-ui-kit-documentation#115
PR Style: ovh/ovh-ui-kit#214
Staging: http://feature_datagrid_customize_staging.ui-kit.ovh//#!/oui-angular/datagrid