-
-
Notifications
You must be signed in to change notification settings - Fork 104
feat: DynamoDB UI query #193
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
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
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. 🚀 New features to boost your workflow:
|
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>
Signed-off-by: seven <zilisheng1996@gmail.com>
There was a problem hiding this 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,
feat: DynamoDB UI query
Refs: #142