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

feat: auto select schema #22160

Merged
merged 1 commit into from
Nov 18, 2022

Conversation

betodealmeida
Copy link
Member

SUMMARY

When choosing a database, if it has only a single schema, auto select it for the user.

I thought of doing the same when the schema has only a single table, but selecting a table triggers a preview and the user might not want to do that.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

N/A

TESTING INSTRUCTIONS

Load any database in SQL Lab that has only one schema.

ADDITIONAL INFORMATION

  • Has associated issue:
  • Required feature flags:
  • Changes UI
  • Includes DB Migration (follow approval process in SIP-59)
    • Migration is atomic, supports rollback & is backwards-compatible
    • Confirm DB migration upgrade and downgrade tested
    • Runtime estimates and downtime expectations provided
  • Introduces new feature or API
  • Removes existing feature or API

@betodealmeida betodealmeida changed the title feature: auto select schema feat: auto select schema Nov 17, 2022
@codecov
Copy link

codecov bot commented Nov 18, 2022

Codecov Report

Merging #22160 (1bd9143) into master (83d990d) will decrease coverage by 0.00%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master   #22160      +/-   ##
==========================================
- Coverage   66.99%   66.99%   -0.01%     
==========================================
  Files        1833     1833              
  Lines       69935    69936       +1     
  Branches     7571     7572       +1     
==========================================
  Hits        46855    46855              
  Misses      21121    21121              
- Partials     1959     1960       +1     
Flag Coverage Δ
javascript 53.79% <50.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...frontend/src/components/DatabaseSelector/index.tsx 91.17% <50.00%> (-1.37%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@lyndsiWilliams
Copy link
Member

/testenv up

@github-actions
Copy link
Contributor

@lyndsiWilliams Ephemeral environment spinning up at http://34.211.99.17:8080. Credentials are admin/admin. Please allow several minutes for bootstrapping and startup.

@lyndsiWilliams
Copy link
Member

This is awesome, works great in the test env!

@betodealmeida betodealmeida merged commit 17c2bd8 into apache:master Nov 18, 2022
@github-actions
Copy link
Contributor

Ephemeral environment shutdown and build artifacts deleted.

@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 2.1.0 and removed 🚢 2.1.3 labels Mar 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels size/S 🚢 2.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants