Skip to content

Conversation

@deon-sanchez
Copy link
Collaborator

@deon-sanchez deon-sanchez commented Sep 30, 2025

This pull request enhances the ingest settings functionality in the cloud picker UI by introducing a new option to capture table structure during knowledge ingest, improving the layout and usability of the settings panel, and ensuring the frontend settings are synchronized with backend defaults and user settings. The changes also refactor how settings are managed and displayed, making the UI more informative and robust.

New ingest settings feature:

  • Added a tableStructure option to the ingest settings, allowing users to enable capturing table structure during document ingest. This is reflected in the UI and type definitions (frontend/src/components/cloud-picker/ingest-settings.tsx, frontend/src/components/cloud-picker/types.ts, frontend/src/lib/constants.ts). [1] [2] [3]

UI and usability improvements:

  • Refactored the ingest settings panel layout for clarity, grouping settings, using labeled fields, and adding helper text for each option. The embedding model selector is now a disabled dropdown with contextual information, and units are displayed for chunk size and overlap fields (frontend/src/components/cloud-picker/ingest-settings.tsx).
  • Updated the default open state of the ingest settings panel to improve visibility for users (frontend/src/components/cloud-picker/unified-cloud-picker.tsx).

Settings synchronization and management:

  • Ingest settings are now initialized from backend defaults (DEFAULT_KNOWLEDGE_SETTINGS) and synchronized with backend/user settings using React Query and context. Multiple useEffect hooks ensure frontend state stays in sync with backend changes for chunk size, overlap, OCR, picture descriptions, and embedding model (frontend/src/components/cloud-picker/unified-cloud-picker.tsx, frontend/src/lib/constants.ts). [1] [2] [3]

Code cleanup and consistency:

  • Updated imports and removed unused icons to keep the codebase clean and consistent (frontend/src/components/cloud-picker/ingest-settings.tsx).
  • Ensured all ingest-related settings are defined in a single place and used consistently throughout the app (frontend/src/lib/constants.ts).

- Added a new 'tableStructure' setting to the IngestSettings interface and UI.
- Updated the IngestSettings component to include a select input for the embedding model.
- Improved layout and accessibility of input fields and switches.
- Integrated default settings for chunk size, chunk overlap, OCR, picture descriptions, and embedding model from constants.
- Implemented synchronization of settings with backend data using React Query.
@deon-sanchez deon-sanchez self-assigned this Sep 30, 2025
@deon-sanchez deon-sanchez changed the title Enhance Ingest Settings UI and functionality Add ingest settings to cloud connectors Sep 30, 2025
@deon-sanchez deon-sanchez changed the title Add ingest settings to cloud connectors Remove sync from cloud connectors (on assumption that it doesn't work) Sep 30, 2025
@deon-sanchez deon-sanchez changed the title Remove sync from cloud connectors (on assumption that it doesn't work) Add ingest settings to cloud connectors Sep 30, 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.

2 participants