Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature: download results in Excel (XSLX) format (closes #801) #831

Merged
merged 3 commits into from
Feb 16, 2016

Conversation

ninneko
Copy link
Contributor

@ninneko ninneko commented Feb 15, 2016

Add download excel button at query result view.

sheet = book.add_worksheet("result")
logging.info(query_data['columns'])

list = []
Copy link
Member

Choose a reason for hiding this comment

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

list is a builtin function in Python. Better to name it with a different name, maybe columns_list?

@arikfr
Copy link
Member

arikfr commented Feb 15, 2016

@ninneko thanks! This is a great addition.

@arikfr arikfr changed the title #801 Download DataSheets as Excel file Feature: download results in Excel (XSLX) format (closes #801) Feb 15, 2016
@ghost
Copy link

ghost commented Feb 16, 2016

Awesome 😊

yohei.naruse added 2 commits February 16, 2016 10:56
schedule = "{:02d}:00".format(now.hour - 3) maybe be negative value when now.hour < 3.
I've fixed it.
@ninneko
Copy link
Contributor Author

ninneko commented Feb 16, 2016

Thank you for reviews.
I've fix it and the test case.

arikfr added a commit that referenced this pull request Feb 16, 2016
Feature: download results in Excel (XSLX) format (closes #801)
@arikfr arikfr merged commit 65f81c4 into getredash:master Feb 16, 2016
@arikfr
Copy link
Member

arikfr commented Feb 16, 2016

Merged. Thank you for this great feature! (and for fixing the bug in tests)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants