-
-
Notifications
You must be signed in to change notification settings - Fork 73
Ability to export data as csv / xlsx file #499
Conversation
🚚 move tests under integration and adapt to dash.testing
⚗️ draft case
update test case for download
dash_table/index.html
Outdated
<script src='https://cdnjs.cloudflare.com/ajax/libs/react/15.4.2/react-dom.js'></script> | ||
|
||
<!-- <script src='https://cdnjs.cloudflare.com/ajax/libs/react/15.4.2/react.js'></script> | ||
<script src='https://cdnjs.cloudflare.com/ajax/libs/react/15.4.2/react-dom.js'></script> --> |
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.
You can remove the two commented versions of React at this point.
demo/index.html
Outdated
@@ -5,11 +5,15 @@ | |||
</head> | |||
<body> | |||
<div id='root'></div> | |||
<script src='https://cdnjs.cloudflare.com/ajax/libs/react/15.4.2/react.js'></script> | |||
<script src='https://cdnjs.cloudflare.com/ajax/libs/react/15.4.2/react-dom.js'></script> | |||
<!-- <script src='https://cdnjs.cloudflare.com/ajax/libs/react/15.4.2/react.js'></script> |
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.
Same
@@ -316,6 +316,8 @@ interface IDefaultProps { | |||
css: IStylesheetRule[]; | |||
data: Data; | |||
editable: boolean; | |||
export_format: 'csv' | 'xlsx'; |
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.
missing none
return wb; | ||
} | ||
|
||
export function createWorksheet(heading: string[][], data: any[], columnID: string[], exportHeader: string ) { |
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 Data
type in tables/props.ts - sheet_add_json
expects any[]
but both types should be compatible as Data
is resolved as (IDatumObject | any)[]
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.
💃 Well done. Get this one out the door! :)
Closes #313