Skip to content

Conversation

vasunous
Copy link

No description provided.

vinodkiran and others added 14 commits July 20, 2025 17:02
- Added validation to ensure `activeWorkspaceId` is present in user requests for all API key operations (get, create, update, import, delete).
- Updated `getWorkspaceSearchOptions` and `getWorkspaceSearchOptionsFromReq` to throw an error if `workspaceId` is not provided.
- Modified service methods to enforce `workspaceId` as a required parameter for database operations related to API keys.
…aces and services

- Updated various interfaces to make `workspaceId` a mandatory field instead of optional.
- Enhanced assistant and export-import service methods to require `workspaceId` for operations, ensuring proper validation and error handling.
- Modified database entity definitions to reflect the change in `workspaceId` from optional to required.
- Improved error handling in controllers to check for `activeWorkspaceId` before proceeding with requests.
- Updated controllers for credentials, datasets, document stores, evaluations, evaluators, and variables to enforce the presence of `workspaceId`.
- Enhanced error handling to throw appropriate errors when `workspaceId` is not provided.
- Modified service methods to accept `workspaceId` as a mandatory parameter for operations, ensuring consistent validation across the application.
… retrieval

- Modified the runAdditionalEvaluators function to accept workspaceId as a parameter.
- Updated chatflow and flow-config controllers to require workspaceId for fetching chatflows.
- Modified service methods to accept workspaceId as a parameter, ensuring proper context for chatflow retrieval.
@HenryHengZJ
Copy link
Contributor

HenryHengZJ commented Sep 30, 2025

is the intention to merge into #5228 not to the main ? because this PR contains lots of unrelated changes to CORS, mainly coming from #5228

@vasunous
Copy link
Author

is the intention to merge into #5228 not to the main ? because this PR contains lots of unrelated changes to CORS, mainly coming from #5228

Ignore this PR. Will create new one with the right branch.

@vasunous vasunous closed this 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.

3 participants