Closed snopoke closed 1 week ago
Attention: Patch coverage is 69.23077%
with 8 lines
in your changes missing coverage. Please review.
:white_check_mark: All tests successful. No failed tests found.
Files with missing lines | Patch % | Lines |
---|---|---|
apps/files/models.py | 61.11% | 7 Missing :warning: |
apps/assistants/sync.py | 50.00% | 1 Missing :warning: |
:loudspeaker: Thoughts on this report? Let us know!
🚨 Try these New Features:
Description
Refactor file creation from external sources to get more consistency and to check for existing files.
Duplicate files thoughts
We are still going to have duplicate files because when we create a new version of an assistant we duplicate the attached files. One option would be to clear out the 'external_id' on the file when we duplicate it. This would force us to re-upload the file to OpenAI which would assign it a new file ID. Thoughts?