Skip to content

allow clearing database/schema in DatasourceModal#39918

Open
gfody wants to merge 2 commits intoapache:masterfrom
gfody:patch-1
Open

allow clearing database/schema in DatasourceModal#39918
gfody wants to merge 2 commits intoapache:masterfrom
gfody:patch-1

Conversation

@gfody
Copy link
Copy Markdown

@gfody gfody commented May 6, 2026

SUMMARY

quick fix to enable specifying 'null' for database and schema fields to allow clearing them

TESTING INSTRUCTIONS

edit a dataset, clear the database or schema, save

ADDITIONAL INFORMATION

@bito-code-review
Copy link
Copy Markdown
Contributor

bito-code-review Bot commented May 6, 2026

Code Review Agent Run #0c1694

Actionable Suggestions - 0
Review Details
  • Files reviewed - 1 · Commit Range: 261d0a6..261d0a6
    • superset-frontend/src/components/Datasource/DatasourceModal/index.tsx
  • Files skipped - 0
  • Tools
    • Whispers (Secret Scanner) - ✔︎ Successful
    • Detect-secrets (Secret Scanner) - ✔︎ Successful

Bito Usage Guide

Commands

Type the following command in the pull request comment and save the comment.

  • /review - Manually triggers a full AI review.

  • /pause - Pauses automatic reviews on this pull request.

  • /resume - Resumes automatic reviews.

  • /resolve - Marks all Bito-posted review comments as resolved.

  • /abort - Cancels all in-progress reviews.

Refer to the documentation for additional commands.

Configuration

This repository uses Superset You can customize the agent settings here or contact your Bito workspace admin at evan@preset.io.

Documentation & Help

AI Code Review powered by Bito Logo

@dosubot dosubot Bot added the data:dataset Related to dataset configurations label May 6, 2026
@pull-request-size pull-request-size Bot added size/S and removed size/XS labels May 7, 2026
@bito-code-review
Copy link
Copy Markdown
Contributor

bito-code-review Bot commented May 7, 2026

Code Review Agent Run #1f89f9

Actionable Suggestions - 0
Review Details
  • Files reviewed - 1 · Commit Range: 261d0a6..bc223d8
    • superset/datasets/schemas.py
  • Files skipped - 0
  • Tools
    • Whispers (Secret Scanner) - ✔︎ Successful
    • Detect-secrets (Secret Scanner) - ✔︎ Successful
    • MyPy (Static Code Analysis) - ✔︎ Successful
    • Astral Ruff (Static Code Analysis) - ✔︎ Successful

Bito Usage Guide

Commands

Type the following command in the pull request comment and save the comment.

  • /review - Manually triggers a full AI review.

  • /pause - Pauses automatic reviews on this pull request.

  • /resume - Resumes automatic reviews.

  • /resolve - Marks all Bito-posted review comments as resolved.

  • /abort - Cancels all in-progress reviews.

Refer to the documentation for additional commands.

Configuration

This repository uses Superset You can customize the agent settings here or contact your Bito workspace admin at evan@preset.io.

Documentation & Help

AI Code Review powered by Bito Logo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

data:dataset Related to dataset configurations size/S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant