Skip to content

Conversation

Blankll
Copy link
Member

@Blankll Blankll commented Apr 15, 2025

feat: DynamoDB UI query

  • query UI
  • query command rust
  • form validation
  • Conditional input form validation
  • query with partition key
  • query with sortkey
  • scan query
  • display query result
  • pagination

Refs: #142

Copy link

codecov bot commented Apr 15, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 53.27%. Comparing base (a3883ae) to head (40b292d).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #193   +/-   ##
=======================================
  Coverage   53.27%   53.27%           
=======================================
  Files          10       10           
  Lines         107      107           
  Branches       13       13           
=======================================
  Hits           57       57           
  Misses         48       48           
  Partials        2        2           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Signed-off-by: seven <zilisheng1996@gmail.com>
Blankll added 14 commits April 16, 2025 02:24
Signed-off-by: seven <zilisheng1996@gmail.com>
Signed-off-by: seven <zilisheng1996@gmail.com>
Signed-off-by: seven <zilisheng1996@gmail.com>
Signed-off-by: seven <zilisheng1996@gmail.com>
Signed-off-by: seven <zilisheng1996@gmail.com>
Signed-off-by: seven <zilisheng1996@gmail.com>
Signed-off-by: seven <zilisheng1996@gmail.com>
Signed-off-by: seven <zilisheng1996@gmail.com>
Signed-off-by: seven <zilisheng1996@gmail.com>
Signed-off-by: seven <zilisheng1996@gmail.com>
Signed-off-by: seven <zilisheng1996@gmail.com>
Signed-off-by: seven <zilisheng1996@gmail.com>
Signed-off-by: seven <zilisheng1996@gmail.com>
@Blankll Blankll marked this pull request as ready for review April 23, 2025 16:20
@Blankll Blankll requested a review from Copilot April 23, 2025 16:20
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR implements the DynamoDB UI query functionality and enhances query handling across multiple editors and connections. Key changes include updating translation keys and UI elements for reset/save actions, refactoring import paths and API calls to support DynamoDB operations, and adding a new DynamoDB query editor component.

Reviewed Changes

Copilot reviewed 23 out of 23 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/views/setting/components/aigc.vue Updated translation keys for reset/save buttons
src/views/editor/es-editor/index.vue Refactored imports and code formatting; added trailing commas for consistency
src/views/editor/dynamo-editor/index.vue Introduced new DynamoDB query editor UI with dynamic form items and validation
src/views/connect/index.vue Replaced tabbed placeholders with the new DynamoDB editor component
src/views/connect/components/es-connect-dialog.vue Minor formatting and import path updates for ES connections
src/views/connect/components/dynamodb-connect-dialog.vue Added table name input and validation; updated default form data
src/views/backup-restore/components/restore.vue Updated query index lookup syntax
src/views/backup-restore/components/backup.vue Adjusted mapping of Elasticsearch indices for backup functionality
src/store/tabStore.ts Added DynamoIndexOrTableOption type and updated active index option getter
src/store/connectionStore.ts Integrated DynamoDB connection support and refactored index selection logic
src/lang/* Added new translation keys for DynamoDB features
src/datasources/* Introduced dynamoApi and updated ApiClients to support DynamoDB operations
src/components/tool-bar.vue Added new button group for DynamoDB editor and adjusted connection/index selection
src/common/monaco/tokenlizer.ts Removed redundant logging in the QDSL transformer
src-tauri/* Integrated new dynamo_api into the Tauri backend and updated Cargo.toml
Comments suppressed due to low confidence (1)

src/views/backup-restore/components/restore.vue:235

  • [nitpick] The parameter name 'index' in the find callback is ambiguous given that the object also has an 'index' property. Consider renaming the parameter (e.g., to 'esIndex') for improved clarity.
index => index.index === restoreFormData.value.index,

@Blankll Blankll merged commit 7fdfee1 into master Apr 23, 2025
1 check passed
@Blankll Blankll deleted the feat/dynamo-ui branch April 23, 2025 16:22
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.

1 participant