Skip to content
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

INTEGRATIONS: Remove File By Id #340

Open
Gummimundur opened this issue Apr 6, 2023 · 1 comment
Open

INTEGRATIONS: Remove File By Id #340

Gummimundur opened this issue Apr 6, 2023 · 1 comment
Labels
INTEGRATIONS Tests to verify code works end-to-end with external resources

Comments

@Gummimundur
Copy link
Collaborator

Gummimundur commented Apr 6, 2023

The Ask

We need to develop an Integration test to verify Remove File By Id components are working correctly

How to Complete this Task

Here's some steps to complete this deliverable.

How to claim this issue?

  • Post on this issue a comment that says: "I claim this issue"
  • You will be given 24 hours to complete the issue - if no PR is opened within 24 hours someone else will claim it
  • If you have an open PR but is inactive for more than 24 hours from the last review comment we will shutdown your PR
@Gummimundur Gummimundur added the INTEGRATIONS Tests to verify code works end-to-end with external resources label Apr 6, 2023
@Gummimundur Gummimundur added this to the Delete File By Id milestone Apr 6, 2023
@Gummimundur Gummimundur changed the title INTEGRATIONS: Delete File By Id INTEGRATIONS: Remove File By Id Apr 6, 2023
@sulton-max
Copy link
Contributor

Note : we need good workaround to create delay after upload operation before sending request to delete that file. Because Open AI will start to process the file after uploading, thus it won't be available to delete until some time

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
INTEGRATIONS Tests to verify code works end-to-end with external resources
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants