Skip to content

[ENH] SQL: Limit tables in dropdown#7214

Merged
lanzagar merged 6 commits intobiolab:masterfrom
VesnaT:owsql_count
Dec 19, 2025
Merged

[ENH] SQL: Limit tables in dropdown#7214
lanzagar merged 6 commits intobiolab:masterfrom
VesnaT:owsql_count

Conversation

@VesnaT
Copy link
Contributor

@VesnaT VesnaT commented Dec 18, 2025

Issue

Ensure better performance when working with large databases by using a searchable dropdown or textfield.

Description of changes
  • remove the warning message
  • remove the “Download data to local” option
  • add a limit of 1000 tables in dropdown
  • add radio buttons to switch between Table and Custom SQL
Includes
  • Code changes
  • Tests
  • Documentation

@markotoplak markotoplak changed the title SQL: Limit tables in dropdown [ENH] SQL: Limit tables in dropdown Dec 19, 2025
@codecov
Copy link

codecov bot commented Dec 19, 2025

Codecov Report

❌ Patch coverage is 84.00000% with 12 lines in your changes missing coverage. Please review.
✅ Project coverage is 89.02%. Comparing base (9f3d10b) to head (7ae2042).
⚠️ Report is 20 commits behind head on master.

❌ Your patch status has failed because the patch coverage (84.00%) is below the target coverage (95.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #7214      +/-   ##
==========================================
- Coverage   89.03%   89.02%   -0.01%     
==========================================
  Files         335      335              
  Lines       74212    74239      +27     
==========================================
+ Hits        66071    66093      +22     
- Misses       8141     8146       +5     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@lanzagar lanzagar merged commit 9e35194 into biolab:master Dec 19, 2025
26 of 30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants