DataGrid Proptype mitake?

[santaclauze]

react-admin 2.6.0

It seems that the Props types of Datagrid has a mistake.

Datagrid.propTypes = {
...
    currentSort: PropTypes.shape({
        sort: PropTypes.string,
        order: PropTypes.string,
    }).isRequired,
...
}

However if I log this.props, I have currentSort: {field: "id", order: "DESC"}. See how it contains field and not sort.

In some resources I also get a warning: Warning: Failed prop type: The prop currentSortis marked as required inDatagrid, but its value is undefined.

I suggest a PR with :

Datagrid.propTypes = {
...
    currentSort: PropTypes.shape({
        field: PropTypes.string,
        order: PropTypes.string,
    }).isRequired,
...
}
...
Datagrid.defaultProps = {
  currentSort: {},
};

[djhi]

Thanks for reporting! Can you make a PR fixing the proptypes names, removing the isRequired on it but leaving the default as it is?

[santaclauze]

rdy. Closing