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

Bugfix to read more than 100 files inside an uploaded folder #572

Open
wants to merge 3 commits into
base: 1.6
Choose a base branch
from

Conversation

mschweiger90
Copy link

When you upload an asset folder with more than 100 files inside directly in pimcore, only 100 files are saved!

Bugfix to read more than 100 files inside an uploaded folder
@robertSt7
Copy link
Contributor

Hi @mschweiger90 since this is a bugfix could you please change the base branch to 1.4 and use const and let instead of var? Thanks

@mattamon mattamon changed the base branch from 1.x to 1.5 September 2, 2024 08:42
Copy link

sonarcloud bot commented Sep 2, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
E Maintainability Rating on New Code (required ≥ B)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

@mattamon
Copy link
Contributor

mattamon commented Sep 2, 2024

@robertSt7 I was able to change the base branch to 1.5 and changed to const and let.

You can ignore the sonarcloud warnings and the files changed because of the CS fixer

@kingjia90
Copy link
Contributor

Looks like some Chrome limitation Screenshot from 2024-09-13 13-14-40

Is it limited to Chrome v77? Could you please share your current browser version where you encountered this issue?

So the PR looks LGTM on paper, i will give it a try to see if they changed it later on

@kingjia90 kingjia90 changed the base branch from 1.5 to 1.6 October 1, 2024 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants