Skip to content
This repository has been archived by the owner on Jun 4, 2024. It is now read-only.

Change component name to DataTable #187

Merged
merged 3 commits into from
Oct 31, 2018

Conversation

Marc-Andre-Rivet
Copy link
Contributor

For #154.

  • update name to DataTable
  • update usage & tests

Will require a follow up PR in dash-docs to update usage there as well once merged.

- update usage & tests
- bump version
@chriddyp chriddyp temporarily deployed to dash-table-review-pr-187 October 31, 2018 17:03 Inactive
@@ -22,7 +22,7 @@
"build:js-dev": "run-s \"private::build -- --mode development\"",
"build:js-test": "run-s \"private::build -- --config webpack.test.config.js\"",
"build:js-test-watch": "run-s \"private::build -- --config webpack.test.config.js --watch\"",
"build:py": "./extract-meta src/dash-table/Table.js > dash_table/metadata.json && cp package.json dash_table",
"build:py": "./extract-meta src/dash-table/DataTable.js > dash_table/metadata.json && cp package.json dash_table",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

extract prop types from new file

@@ -16,7 +16,7 @@ function isBackEnd(value) {
return ['be', false].indexOf(value) !== -1;
}

export default class Table extends Component {
export default class DataTable extends Component {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

changing class name to be consistent

@@ -26,7 +26,7 @@
app.layout = html.Div(
[
html.Div(id="container", children="Hello World"),
dash_table.Table(
dash_table.DataTable(
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Changing test usage

package.json Outdated
@@ -1,6 +1,6 @@
{
"name": "dash-table",
"version": "3.1.0rc16",
"version": "3.1.0-rc17",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

bump version

@chriddyp chriddyp temporarily deployed to dash-table-review-pr-187 October 31, 2018 17:14 Inactive
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants