Skip to content

Delete knowledge#44

Merged
deon-sanchez merged 14 commits intomainfrom
delete-knowledge
Sep 18, 2025
Merged

Delete knowledge#44
deon-sanchez merged 14 commits intomainfrom
delete-knowledge

Conversation

@deon-sanchez
Copy link
Copy Markdown
Collaborator

This pull request introduces several improvements to the frontend, focusing on enhancing user experience and robustness in file uploads, cloud connector navigation, and input handling. The main changes include improved loading and error states in the KnowledgeDropdown component, a new reusable confirmation dialog, and more resilient handling of API responses.

User Experience & Feedback Improvements:

  • Added comprehensive loading states and feedback in the KnowledgeDropdown component, including a spinner and context-aware button text during uploads, folder/S3 processing, and cloud connector navigation. This prevents user actions during ongoing operations and improves clarity. [1] [2] [3]
  • Replaced direct search query invalidation with a custom knowledgeUpdated event to trigger refreshes after uploads or ingestion, centralizing refresh logic and reducing unnecessary queries. [1] [2] [3] [4] [5] [6]

Robustness & Error Handling:

  • Improved file upload and ingestion error handling: now checks for both unified and simple API response formats, validates ingestion status, and provides clearer error messages, especially for Langflow component failures.
  • Enhanced cloud connector navigation by adding a loading state and error handling when routing to cloud upload pages, ensuring feedback is provided if navigation is delayed or fails.

Component & Code Quality Improvements:

  • Added a new reusable DeleteConfirmationDialog component for consistent confirmation dialogs across the app.
  • Improved the Input component for better style consistency and code readability, including handling of custom class names and placeholder visibility. [1] [2] [3]
  • Made the nudges API response handling more robust, falling back to defaults if the response is missing or malformed. [1] [2]

- Added a new `DeleteConfirmationDialog` component for confirming deletions.
- Updated `KnowledgeDropdown` to include a loading state and improved user feedback during file operations.
- Enhanced the search page to support bulk deletion of documents with confirmation dialog.
- Integrated event dispatching for knowledge updates after file operations.
- Refactored various components for better readability and maintainability.
deon-sanchez and others added 12 commits September 17, 2025 17:09
- Added a delete confirmation dialog to the KnowledgeActionsDropdown component.
- Integrated useDeleteDocument mutation for handling document deletion.
- Updated the dropdown to pass the filename prop for targeted deletions.
- Enhanced user feedback with success and error messages upon deletion.
- Implemented bulk selection and deletion functionality in the Knowledge page.
- Added a confirmation dialog for bulk deletions, providing user feedback on success or failure.
- Updated AgGrid configuration to support multiple row selection and checkbox functionality.
- Styled checkboxes for better visibility and user experience.
- Refactored related components to accommodate new bulk actions.
- Removed the clear selection functionality and associated UI elements from the SearchPage component.
- Simplified the bulk actions bar by integrating a delete button and a sync button with alert functionality.
- Adjusted button styles for consistency and improved user experience.
- Enhanced the search input area for better accessibility and usability.
- Updated the file selection UI to allow clicking on file names for selection, enhancing user interaction.
- Changed AgGrid configuration to enable row selection with clicks, improving usability.
- Removed unnecessary row click event handling to streamline the selection process.
- Added TODO comments for implementing rename and sync functionalities in the KnowledgeActionsDropdown.
- Adjusted column definitions in the SearchPage component to improve layout with flex and minWidth properties.
- Commented out the sync button in the SearchPage for future implementation, enhancing code clarity.
- Introduced a new ChunksPage component to display chunks of a selected file, enhancing the knowledge management functionality.
- Updated the SearchPage component to navigate to the ChunksPage when a file is selected, improving user flow.
- Removed the previous file selection logic and integrated a more streamlined approach for chunk display.
- Enhanced UI elements for better user experience and accessibility.
@deon-sanchez deon-sanchez changed the title Delete knowledge (DO NOT MERGE) Delete knowledge Sep 18, 2025
@deon-sanchez deon-sanchez merged commit a05e71b into main Sep 18, 2025
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.

3 participants