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.

Sign up for GitHub

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

Jump to bottom

feat(sqllab) Add a configuration option to disable data preview #14768

Closed
wants to merge 9 commits into from
Closed

feat(sqllab) Add a configuration option to disable data preview #14768

wants to merge 9 commits into from

Commits on May 22, 2021

  1. feat(sqllab) Add a configuration option to disable data preview 

    This setting is per-database in the "extra" JSON data.
    @akedrou
    akedrou committed May 22, 2021
    Configuration menu
    Copy the full SHA
    6cc44c9 View commit details
    Browse the repository at this point in the history

Commits on Feb 24, 2022

  1. Initial commit to fix merge issues, still need to go through and make… 

    … sure the code is compatible with master
    @cccs-Dustin
    cccs-Dustin committed Feb 24, 2022
    Configuration menu
    Copy the full SHA
    2cfe589 View commit details
    Browse the repository at this point in the history

  2. Added in modifications to allow for the old PR feature to be compatib… 

    …le with master
    @cccs-Dustin
    cccs-Dustin committed Feb 24, 2022
    Configuration menu
    Copy the full SHA
    ad29be6 View commit details
    Browse the repository at this point in the history

Commits on Feb 25, 2022

  1. Fixed some issues that were caused by the merge conficts

    @cccs-Dustin
    cccs-Dustin committed Feb 25, 2022
    Configuration menu
    Copy the full SHA
    bf2174b View commit details
    Browse the repository at this point in the history

  2. Modified the code to fix the problem where some of the tests were fai… 

    …ling
    @cccs-Dustin
    cccs-Dustin committed Feb 25, 2022
    Configuration menu
    Copy the full SHA
    b748285 View commit details
    Browse the repository at this point in the history

  3. Ran pre-commit hook to remove any issues it would pick up

    @cccs-Dustin
    cccs-Dustin committed Feb 25, 2022
    Configuration menu
    Copy the full SHA
    8eb76a9 View commit details
    Browse the repository at this point in the history

  4. Fixed a pylint error where a line was too long

    @cccs-Dustin
    cccs-Dustin committed Feb 25, 2022
    Configuration menu
    Copy the full SHA
    fb1343b View commit details
    Browse the repository at this point in the history

  5. Fixed the test so it no longer fails, since it was looking for 7 chec… 

    …kboxes and tooltips but there are now 8
    @cccs-Dustin
    cccs-Dustin committed Feb 25, 2022
    Configuration menu
    Copy the full SHA
    f91c691 View commit details
    Browse the repository at this point in the history

  6. Fixed the test so it no longer fails, since it was looking for 7 chec… 

    …kboxes and tooltips but there are now 8
    @cccs-Dustin
    cccs-Dustin committed Feb 25, 2022
    Configuration menu
    Copy the full SHA
    8459ad6 View commit details
    Browse the repository at this point in the history