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

Always show 'Download Notebook' button, but disable it when not available #799

Merged
merged 4 commits into from
Dec 8, 2022

Conversation

mknowlton89
Copy link
Collaborator

@mknowlton89 mknowlton89 commented Dec 6, 2022

Features and Changes

Description of Feature
On experiment results, we currently hide the Download Notebook option in the MoreMenu dropdown by default. It's only enabled if the user defines a Jupyter Query Runner for their data source.

This is a really cool feature that most users don't know about. Instead of hiding it, we can still show the option, but grayed out with a tooltip explaining how to enable it.

Dependencies

N/A

Testing

  • Go to an experiment in an organization that does not have a query runner set up and ensure that the button is visible, it's disabled, and the tooltip is display as intended.
  • Create a query runner (or test via an organization that has a query runner) and ensure the button is visible, enabled, and no tooltip is displayed.

Screenshots

Screen Shot 2022-12-06 at 2 43 36 PM

Screen Shot 2022-12-06 at 2 43 26 PM

…e it if the organization is not set up for it yet, and also add tooltip with brief guidance on what this button does and how to get set up.
@mknowlton89 mknowlton89 marked this pull request as ready for review December 7, 2022 16:43
Copy link
Contributor

@bttf bttf left a comment

Choose a reason for hiding this comment

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

👍🏾

@mknowlton89 mknowlton89 merged commit 03456d3 into main Dec 8, 2022
@mknowlton89 mknowlton89 deleted the michaelk/show-export-to-notebook branch December 8, 2022 19:30
lukesonnet pushed a commit that referenced this pull request Dec 12, 2022
…able (#799)

* Refactor the 'Download Notebook' button to always display, but disable it if the organization is not set up for it yet, and also add tooltip with brief guidance on what this button does and how to get set up.
lukesonnet pushed a commit that referenced this pull request Dec 12, 2022
…able (#799)

* Refactor the 'Download Notebook' button to always display, but disable it if the organization is not set up for it yet, and also add tooltip with brief guidance on what this button does and how to get set up.
lukesonnet pushed a commit that referenced this pull request Dec 12, 2022
…able (#799)

* Refactor the 'Download Notebook' button to always display, but disable it if the organization is not set up for it yet, and also add tooltip with brief guidance on what this button does and how to get set up.
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