-
Notifications
You must be signed in to change notification settings - Fork 8.6k
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: DocumentAddByFileApi miss data_source_type field but there is a mandatory value check #12273
fix: DocumentAddByFileApi miss data_source_type field but there is a mandatory value check #12273
Conversation
Please fix the error in CI. |
pls fix the python style check @lyfuci |
@JohnJyong I am quite a rookie for this project, and after running the ruff format command, I found that a comma was missing at the end, and adjusted. After checking the DocumentUpdateByFileApi, I noticed the same problem of the issue relatted to this PR exists and has been fixed as well. |
Just run |
@laipz8200 thanks,I use "ruff format" command seems work, anything wrong just tell me. |
Summary
Fixed a bug in DocumentAddByFileApi where the data_source_type field and its corresponding value were missing when constructing the data_source. Fixes #12272
Screenshots
before:
after:
Checklist
Important
Please review the checklist below before submitting your pull request.
dev/reformat
(backend) andcd web && npx lint-staged
(frontend) to appease the lint gods