-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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
fix: Implement get_or_create_default_folder for user folder management #6090
Conversation
…lt folder exists for users
…r_create_default_folder for user folder creation
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.
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
Comments suppressed due to low confidence (1)
src/backend/base/langflow/initial_setup/setup.py:780
- Ensure that the new behavior introduced by the get_or_create_default_folder function is covered by tests.
async def get_or_create_default_folder(session: AsyncSession, user_id: UUID) -> FolderRead:
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.
lgtm
CodSpeed Performance ReportMerging #6090 will improve performances by 10.79%Comparing Summary
Benchmarks breakdown
|
…dability and maintainability by chaining actions on page elements instead of using separate lines for each action
…ow into fix-psql-folder-error
Introduce a new function to ensure the default folder exists for users, replacing the previous method. This change streamlines folder creation logic across the application.
Related #6054