Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

feat(upload): add documentation for new setting#2445

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
Tyki wants to merge1 commit intostrapi:main
base:main
Choose a base branch
Loading
fromTyki:main

Conversation

Tyki
Copy link

@TykiTyki commentedMar 27, 2025•
edited by pwizla
Loading

What does it do?

This pull request add a new settings to limit the concurrency of uploads to 1 in the media library.

It also fix a small issue when the file size is checked only after that images optimizations are applied. But it means that, whatever the image size is, sharp will try to optimize it. On low ram environments, it leads to OOM.

How to test it?

Add the new setting in the configuration for the upload plugin: limitConcurrentUpload

  upload: {    config: {      limitConcurrentUploads: true    }  },

and then try to upload multiple assets in the same time

Related issue(s)/PR(s)

strapi/strapi#23253

@strapi-cla
Copy link

strapi-cla commentedMar 27, 2025•
edited
Loading

CLA assistant check
All committers have signed the CLA.

@vercelVercel
Copy link

vercelbot commentedMar 27, 2025•
edited
Loading

The latest updates on your projects. Learn more aboutVercel for Git ↗︎

NameStatusPreviewCommentsUpdated (UTC)
documentation✅ Ready (Inspect)Visit Preview💬Add feedbackMar 27, 2025 8:46pm

@pwizla
Copy link
Collaborator

Thank you,@Tyki!
The PR is approved, but I'll wait for the corresponding strapi/strapi PR to be merged before merging the present PR (hence theflag: merge pending release—it will make some test fail to ensure it's not merged prematurely)

@pwizlapwizla added the temp - port to docs-next(Temporary label) The content of this PR should be ported to docs-next.strapi.io labelApr 29, 2025
@pwizlapwizlaforce-pushed themain branch 2 times, most recently from25723df to63466f0CompareJune 27, 2025 15:32
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@meganelachenymeganelachenyAwaiting requested review from meganelachenymeganelacheny is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@pwizlapwizla

Labels
flag: merge pending releasePending Strapi Releasepr: choresource: CMStemp - port to docs-next(Temporary label) The content of this PR should be ported to docs-next.strapi.io
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

3 participants
@Tyki@strapi-cla@pwizla
[8]ページ先頭
©2009-2025 Movatter.jp