Skip to content

Fix database UI query issues - #613

Open
DhruvShah-Dev wants to merge 1 commit into
TabularisDB:mainfrom
DhruvShah-Dev:fix/database-ui-query-fixes
Open

Fix database UI query issues#613
DhruvShah-Dev wants to merge 1 commit into
TabularisDB:mainfrom
DhruvShah-Dev:fix/database-ui-query-fixes

Conversation

@DhruvShah-Dev

@DhruvShah-Dev DhruvShah-Dev commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Use the live selected database list in the object palette and align its multi-database branch with the sidebar.
  • Make query error text selectable and add an explicit copy action.
  • Render MySQL/MariaDB JSON update values as JSON text instead of CAST(... AS JSON), which MariaDB rejects.

Closes #591.
Closes #608.
Closes #610.

Validation

  • pnpm.cmd vitest run tests/components/modals/ErrorModal.test.tsx tests/utils/quickNavigator.test.ts
  • pnpm.cmd typecheck
  • pnpm.cmd lint
  • pnpm.cmd build
  • git diff --check
  • cargo test mysql_json_update_value --manifest-path src-tauri/Cargo.toml --lib --no-run
  • GitHub CI test job passed

@DhruvShah-Dev
DhruvShah-Dev force-pushed the fix/database-ui-query-fixes branch from 23b4526 to ba58c90 Compare August 8, 2026 23:43
@kilo-code-bot

kilo-code-bot Bot commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (5 files)
  • src-tauri/src/drivers/mysql/mod.rs
  • src-tauri/src/drivers/mysql/tests.rs
  • src/components/modals/ErrorModal.tsx
  • src/components/modals/QuickNavigatorModal.tsx
  • tests/components/modals/ErrorModal.test.tsx

Reviewed by step-3.7-flash · Input: 84.7K · Output: 23.1K · Cached: 2.2M

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

Labels

None yet

Projects

None yet

1 participant