forked from apache/superset
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Filters #1
Open
simcha90
wants to merge
191
commits into
master
Choose a base branch
from
filters
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* feat: SIP-34 explore save modal * using a const for the session storage key * backend changes * minor tweaks * more tweaks * radio cosmetics * styles * fix tests * CreatableSelect\! * Fix cypress & lint * fix unit * lint
* chore: wiring ControlLabel to a new FormLabel Creating new simple <FormLabel /> component and wiring all <label> and react-bootstrap.ControlLabel towards it. FormLabel becomes a pivotal point that can be altered to point to AntD when we're ready. * lint * ViewportControl * addressing comments
…che#10402) * Add fade transition to bottom border on navbar menu items * lint fix * use theme variables * fix spec
Flaky tests hurt everyone's cognitive load. Let's deactivate for now and revisit.
As the label change from RUN to STOP, the row of button shifts left/right. This fixes the width so that it doesn't jitter.
* style: SIP-34 labels Aligning with SIP-34 designs for labels/pills. Going uppercase fixes the padding issues we had before, and the rounder pills look better. Cleaning up some CSS in the process * lint
* fix: equals operator in filter box * fix time range endpoint schema * fix test * bump packages * fix tests * lint
* fix(permissions): alpha role is inconsistent * reverse and allow Alpha to access manager menu * Bump FAB to 3.0.1rc1 to include del permission fix * add docs, tests and UPDATING * EOL * Fix query view for Alpha
Signed-off-by: Stuart Hu <[email protected]>
* migrated LanguagePicker.jsx to tsx * Migrated Menu.jsx to tsx * migrated MenuObject.jsx to tsx * migrated NewMenu.jsx to tsx * Migrated UserMenu.jsx to tsx * removed unnecessary export from UserMenu * added language definition in LanguagePicker * removed unnecessary exports from Menu.tsx * used typeof guard for childs * changed LanguageProps to Languages * removed unnecessary type casting * fixed linting errors
* docs: pointers to plugins blog post * fix build
* fix: support non-numeric columns in pivot table * bump package and add unit tests * mypy
* Handle ROW data stored as string * Use destringify * Fix mypy * Fix mypy with cast * Bypass pylint
* implemented updates to alerting * fixed imports and teardown * changed unittest to pytest conventions * add app_context to tests Co-authored-by: Jason Davis <@dropbox.com>
* remove conflicts with csv upload * revert StringField * change description * remove redundant space * apply string approach
* fix(log): log crashes if expired or not authenticated * add auth to log endpoint
* fix(log): log crashes if expired or not authenticated * fix lint and rison * add tests * more tests * perm fix * fix test not found * JS lint * fix Jest test
…ache#10459) * adding jest-enzyme * enzymeify lots of assertions * types for jest-enzyme
…he#10707) * refactor(database): use SupersetResultSet on SqlaTable.get_df() * lint * change cypress test
* fix(jinja): extract form_data from json body * add test * disable test for presto
Co-authored-by: bogdan kyryliuk <[email protected]>
…t Superset (apache#10649) * getting rid of weird focus/active outline ring * Buttons... buttons _everywhere_ * linting * Nixing views/CRUD/dataset/Button component * fixing 2 typing errors * fixing more TS errors * prefer src path for include * one more real button, one less CSS class * one more "button" to "Button" * Published Status is now a proper clickable Label * nixing the CRUD button again * touching up stories, with SupersetButton story * SIP-34 button colors * adding polished package to mix colors * updating button colors to match Superset theme * abstracting away from bootstrap-specific props (might pivot libraries soon!) * more abstraction from bsStyle/bsSize props * exchanging styles for a prop * linting * restoring feature flag to stock * using src alias * last <button> replacement * this classname would never be applied * more linting action * fixing unsupported bsSize 'medium', and cta typing error * more cta action * unnecessary styles * errant bsSize prop * cleanup * tweaks to make new New button work * Linting * fixing a couple tests * fixing theme based test failure * margin tweak for NEW button * another fixed test * another fixed test * fixing two more tests * fixing last broken tests. * always be linting * Adding tertiary/dashed buttons * cleaning up QueryAndSave buttons * fixing "link" button styles * fixing/updating link button styles * cta buttons on Modal component * linting. * exporting button story knobs, making ALL knobs safe for export. * capitalizing a file... no big whoop * Basic button tests * renaming button - temporarily * renaming file to fix capitalization issue * passing theme through to a difficult popover. * fixin' a newly busted unit test * lint fixin' * oops, shouldn't have changed this prop! * adding a dive() to themedShallow, and fixing a cypress/jest test * addressing lint stuff * touching up stories, with SupersetButton story * SIP-34 button colors * updating button colors to match Superset theme * abstracting away from bootstrap-specific props (might pivot libraries soon!) * linting * restoring feature flag to stock * cleanup * Linting * renaming button - temporarily * renaming file to fix capitalization issue * oops, shouldn't have changed this prop! * adding a dive() to themedShallow, and fixing a cypress/jest test * addressing lint stuff * nixing new modal button * Fixing another popover/button issue that should break cypress * lint ✨ * passing classNames through to new button (should fix some tests) * cleaning unused classes, making cypress tests use data attrs * fixin' the test * fixing another class-based test with data-test attr * no longer passing theme as prop to buttons in popovers... themeprovider is better * outline/border tweaks!
* style: rename 'Source' top nav item to 'Data' * Fix tests * tables -> datasets * rename datasource -> dataset * lint * more renaming * use npm ci
…he#10706) * fix execute oracle DML statement bug in sqllab when i execute oracle sql statements like update in SQLLAB, get "oracle error: not a query" error. Refer https://www.python.org/dev/peps/pep-0249/, superset old version use `cursor.description` ,because this attribute will be None for operations that do not return rows or if the cursor has not had an operation invoked via the .execute*() method yet. * Apply suggestions from code review Co-authored-by: Ville Brofeldt <[email protected]> * Update oracle.py * Update oracle.py * Update oracle.py * Apply suggestions from code review Co-authored-by: Ville Brofeldt <[email protected]> * Update oracle.py * Update superset/db_engine_specs/oracle.py Co-authored-by: Ville Brofeldt <[email protected]> Co-authored-by: Ville Brofeldt <[email protected]>
* chore: bump pandas an pyarrow * remove unsupported kwargs * fix types * move pandas back to 1.0 branch due to possible MultiIndex regression
…pache#10728) * Supporting compound coponents, adding stories * Adding a coupe o' knobs just for fun.
…10747) * removing two duplicate devdependencies - leaving dependencies * bumping shuffled dependencies, making fsevents non-optional * bumping @babel/cli * Another bump
* Log cache keys to the logs * Add tests * Use separate table for the cache keys * Add migration for the cache lookup table Co-authored-by: bogdan kyryliuk <[email protected]>
* added new tables for alerting refractor * reformatted inheritance structure * added workflow for updated framework * added suggested changes * cleaned up changes * added obervations to alert table to enable view * added comments * added requested changes * fix tests * added styling changes * mypy * added requested changes * updated operator logic * requested changes, 1 validator, styling changes * refactored tests * fix test alert workflow * fixed create_alert in test Co-authored-by: Jason Davis <@dropbox.com>
* Updated Docker local install instructions Updated Docker local install instructions with more detail, structure, and additional info for Linus and Windows users. * Updated with note on Windows environment
Fix issue where dependencies listed in ./docker/requirements-local.txt are not installed
Co-authored-by: Jason Davis <@dropbox.com>
* initial commit * Change reference to new echarts plugin * initial commit * Change reference to new echarts plugin * moving dependencies to plugin * adding pie chart * rename piechart key * lint * Bump packages and implement echarts colors * remove pie for now Co-authored-by: Evan Rusackas <[email protected]>
Co-authored-by: jagadish <[email protected]>
* feat(database): POST, PUT, DELETE API endpoints * post tests * more tests * lint * lint * debug ci * fix test * fix test * fix test * fix test * fix test * fix test * cleanup * handle db connection failures * lint * skip hive and presto for connection fail test * fix typo
* Fix precommit hook for docs/installation.rst * Fix underline too short
* fix: re-installing local superset in cache image * Fixing webpack
Co-authored-by: John Bodley <[email protected]>
* labels with onClick don't need .pointer. No labels need `m-r-5` * making Timer a proper Label * a little linting * addresing (helpful, thanks) comment
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
SUMMARY
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
TEST PLAN
ADDITIONAL INFORMATION