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

[Components] zip_archive_api #11789

Open
pipedream-component-development opened this issue May 1, 2024 · 4 comments · May be fixed by #11827
Open

[Components] zip_archive_api #11789

pipedream-component-development opened this issue May 1, 2024 · 4 comments · May be fixed by #11827
Assignees
Labels
triaged For maintainers: This issue has been triaged by a Pipedream employee

Comments

@pipedream-component-development
Copy link
Collaborator

zip_archive_api

URLs

@dylburger dylburger added the triaged For maintainers: This issue has been triaged by a Pipedream employee label May 1, 2024
@lcaresia lcaresia self-assigned this May 6, 2024
@lcaresia lcaresia moved this from To Do to Doing in Component (Source and Action) Backlog May 6, 2024
@lcaresia lcaresia linked a pull request May 6, 2024 that will close this issue
@lcaresia lcaresia moved this from Doing to Ready for PR review in Component (Source and Action) Backlog May 6, 2024
@GTFalcao GTFalcao moved this from Ready for PR review to In Review in Component (Source and Action) Backlog May 6, 2024
@GTFalcao GTFalcao moved this from In Review to Ready for QA in Component (Source and Action) Backlog May 6, 2024
@GTFalcao
Copy link
Collaborator

GTFalcao commented May 6, 2024

@vunguyenhung @malexanderlim I've commented on a usability concern on this issue's PR - it seems this API supports file uploading (via form-data), if that's the case, our components should probably support that as well.

@malexanderlim
Copy link
Contributor

Thanks @GTFalcao . @lcaresia , could you implement to accept files both via File URL / using Form Data / application/octet-stream as well, to maintain maximum flexibility here?

@vunguyenhung vunguyenhung moved this from Ready for QA to In QA in Component (Source and Action) Backlog May 7, 2024
@vunguyenhung vunguyenhung moved this from In QA to Changes Required in Component (Source and Action) Backlog May 7, 2024
@vunguyenhung
Copy link
Collaborator

Hello everyone, I have tested this PR and there're some test cases failed or needed improvement.

Please check the test report below for more information
https://vunguyenhung.notion.site/Components-zip_archive_api-3f0dbfdd65c94f9e9914d4669acd6ec2

@lcaresia lcaresia moved this from Changes Required to Ready for PR review in Component (Source and Action) Backlog May 17, 2024
@luancazarine luancazarine moved this from Ready for PR review to In Review in Component (Source and Action) Backlog May 17, 2024
@luancazarine luancazarine moved this from In Review to Ready for QA in Component (Source and Action) Backlog May 17, 2024
@vunguyenhung vunguyenhung moved this from Ready for QA to In QA in Component (Source and Action) Backlog May 20, 2024
@vunguyenhung vunguyenhung moved this from In QA to Changes Required in Component (Source and Action) Backlog May 20, 2024
@vunguyenhung
Copy link
Collaborator

Hello everyone, I have tested this PR and there're some test cases failed or needed improvement.

Please check the test report below for more information
https://vunguyenhung.notion.site/Components-zip_archive_api-3f0dbfdd65c94f9e9914d4669acd6ec2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triaged For maintainers: This issue has been triaged by a Pipedream employee
Projects
Development

Successfully merging a pull request may close this issue.

6 participants