-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Misc changes to codebase back ported from internal fork #5129
Conversation
@arikfr As someone also maintaining an internal fork, can you share any plans yet what the mode of maintenance for the OSS version of Redash will be? |
@jezdez right now things are blocked as we're "regrouping", but things will remain similar to what they were until now: 2-4 stable releases every year and regular cadence of ongoing development/code reviews. (I hope this answers the question, happy to clarify if needed) |
* Set corejs version in .babelrc so Jest doesn't complain. * Rewrite services/routes in TypeScript. * Add TypeScript definitions for DialogComponent. * Make image paths more portable * Add current route context and hook. * Make EmptyState more flexible by being able to pass in getSteps function. * Rewrite ItemsList in TypeScript. * Introduce the possibility to add custom sorters for a column. * Rearrange props to be friendly to TypeScript. * Type definitions for NotificationApi. * Use Databricks query editor components for databricks_internal type of query runner. * URL Escape password in Alembic configuration. * Compare types in migrations.
What type of PR is this? (check all applicable)
Description
This is lots of small changes from the internal fork at Databricks. Most of them are unrelated to each other, but as they were "vetted" and tested in the forked version, it seemed safe to just group them all together.
Most of them aren't changing functionality but rather extending it.
See commit messages for descriptions.