Skip to content

[Task]: Verify we are using Flysystem everywhere #254

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

Merged
merged 5 commits into from
Jul 24, 2024

Conversation

lukmzig
Copy link
Contributor

@lukmzig lukmzig commented Jul 11, 2024

Changes in this pull request

Resolves #244

@lukmzig lukmzig added the Task label Jul 11, 2024
@lukmzig lukmzig marked this pull request as draft July 11, 2024 14:39
@lukmzig lukmzig marked this pull request as ready for review July 24, 2024 09:09
mattamon and others added 2 commits July 24, 2024 10:09
* Refactor CSV Download to jobrunId based

* Apply php-cs-fixer changes

* Updating storage

* Apply php-cs-fixer changes

* Try to use fly system for zip

* Apply php-cs-fixer changes

* fix: STAN

* make zip download sequential

* Apply php-cs-fixer changes

* remove unused storage service

* make csv export sequential

* Apply php-cs-fixer changes

* fixes

* add csv export changes

* Apply php-cs-fixer changes

* add zip upload changes

* Apply php-cs-fixer changes

* fix: qodana

* fix: qodana

* Apply php-cs-fixer changes

* fix: STAN

* update download service

* Apply php-cs-fixer changes

* fix: qodana and remove unused ENUMs

* add review changes

---------

Co-authored-by: mattamon <[email protected]>
Co-authored-by: lukmzig <[email protected]>
Co-authored-by: lukmzig <[email protected]>
Copy link

@lukmzig lukmzig merged commit b9bae93 into 1.x Jul 24, 2024
13 checks passed
@lukmzig lukmzig deleted the 244-task-verify-we-are-using-flysystem-everywhere branch July 24, 2024 10:39
@github-actions github-actions bot locked and limited conversation to collaborators Jul 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Task]: Verify we are using Flysystem everywhere
2 participants