-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Move sizeLimit one level above in upload provider #1133
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
Move sizeLimit one level above in upload provider #1133
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
I added a new migration file explaining:
What do you think? |
gpene
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
pwizla
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you very much and congrats for your very first PR to our documentation repo, @Marc-Roig! 🥳
Good job.
I added some suggestions, those are mostly formatting-related. Happy to help if you have any questions. 🤗
...er-docs/latest/update-migration-guides/migration-guides/v4/migration-guide-4.3.x-to-4.4.x.md
Outdated
Show resolved
Hide resolved
...er-docs/latest/update-migration-guides/migration-guides/v4/migration-guide-4.3.x-to-4.4.x.md
Outdated
Show resolved
Hide resolved
...er-docs/latest/update-migration-guides/migration-guides/v4/migration-guide-4.3.x-to-4.4.x.md
Outdated
Show resolved
Hide resolved
...er-docs/latest/update-migration-guides/migration-guides/v4/migration-guide-4.3.x-to-4.4.x.md
Outdated
Show resolved
Hide resolved
...er-docs/latest/update-migration-guides/migration-guides/v4/migration-guide-4.3.x-to-4.4.x.md
Outdated
Show resolved
Hide resolved
...er-docs/latest/update-migration-guides/migration-guides/v4/migration-guide-4.3.x-to-4.4.x.md
Outdated
Show resolved
Hide resolved
...er-docs/latest/update-migration-guides/migration-guides/v4/migration-guide-4.3.x-to-4.4.x.md
Outdated
Show resolved
Hide resolved
…uides/v4/migration-guide-4.3.x-to-4.4.x.md Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com>
…uides/v4/migration-guide-4.3.x-to-4.4.x.md Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com>
…uides/v4/migration-guide-4.3.x-to-4.4.x.md Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com>
…uides/v4/migration-guide-4.3.x-to-4.4.x.md Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com>
…uides/v4/migration-guide-4.3.x-to-4.4.x.md Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com>
The base branch was changed.
|
Sorry for the wait , this will now be ready to merge in 4.6.1 . So we will have to merge that PR first |
pwizla
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM now. Thanks again, Marc!
* user-permissions plugin: document how to add Patreon as an Oauth provider * add server webhooks populateRelations config variable * Update docs/developer-docs/latest/setup-deployment-guides/configurations/required/server.md Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com> * Auto stash before merge of "dev/webhook-populated-relations" and "origin/dev/webhook-populated-relations" * add 4.6.1 migration guide * Update docs/developer-docs/latest/setup-deployment-guides/configurations/required/server.md Co-authored-by: Pierre Noël <petersg83@users.noreply.github.com> * Update docs/developer-docs/latest/update-migration-guides/migration-guides/v4/migration-guide-4.5.1-to-4.6.1.md Co-authored-by: Pierre Noël <petersg83@users.noreply.github.com> * Update docs/developer-docs/latest/update-migration-guides/migration-guides/v4/migration-guide-4.5.1-to-4.6.1.md Co-authored-by: Pierre Noël <petersg83@users.noreply.github.com> * Update docs/developer-docs/latest/update-migration-guides/migration-guides/v4/migration-guide-4.5.1-to-4.6.1.md Co-authored-by: Pierre Noël <petersg83@users.noreply.github.com> * improve explanation of webhooks populate relations usage * Update filtering-locale-publication.md (#1441) for this pull strapi/strapi#15046 * Fix deployment instructions for Azure CLI # Change 1: Update Node runtime argument of ```az webapp create``` Running the docs as-is produces an error: ```... Linux Runtime 'node|10.14' is not supported.Run 'az webapp list-runtimes --os-type linux' to cross check``` The correct parameter today is ```bash --runtime "node|14-lts" ``` # Change 2: Update IP address block arguments of ```az mysql server firewall-rule create``` Running the docs as-is produces an error: ```... the following arguments are required: --start-ip-address, --end-ip-address``` Based on [the most recent docs](https://learn.microsoft.com/en-us/cli/azure/mysql/server/firewall-rule?view=azure-cli-latest), the correct parameters to define the allowed IP blocks are: ```bash --start-ip-address 0.0.0.0 --end-ip-address 0.0.0.0 ``` * Change Node 14 to Node 16 https://azure.microsoft.com/en-us/updates/community-support-for-node-14-lts-is-ending-on-30-april-2023/ * Update docs/developer-docs/latest/plugins/users-permissions.md * Update docs/developer-docs/latest/plugins/users-permissions.md * Fix broken links reported in issue #1423 (#1484) * Fix broken links reported in issue #1423 * Fix BE custom. > middlewares link * Update quick-start.md (#1478) * Move sizeLimit one level above in upload provider (#1133) * move sizeLimit one level above in upload provider * add sizeLimit deprecation warning * minor change * minor change * checkFileSize upload provider * Update docs/developer-docs/latest/update-migration-guides/migration-guides/v4/migration-guide-4.3.x-to-4.4.x.md Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com> * Update docs/developer-docs/latest/update-migration-guides/migration-guides/v4/migration-guide-4.3.x-to-4.4.x.md Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com> * Update docs/developer-docs/latest/update-migration-guides/migration-guides/v4/migration-guide-4.3.x-to-4.4.x.md Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com> * Update docs/developer-docs/latest/update-migration-guides/migration-guides/v4/migration-guide-4.3.x-to-4.4.x.md Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com> * Update docs/developer-docs/latest/update-migration-guides/migration-guides/v4/migration-guide-4.3.x-to-4.4.x.md Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com> * Update docs/developer-docs/latest/update-migration-guides/migration-guides/v4/migration-guide-4.3.x-to-4.4.x.md Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com> * Update docs/developer-docs/latest/update-migration-guides/migration-guides/v4/migration-guide-4.3.x-to-4.4.x.md Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com> * move migration steps to 4.5.1 to 4.6.1 --------- Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com> * update localhost link * Connect can not be used for media attributes (#1490) * connect can not be used for meida attributes * change warning to caution --------- Co-authored-by: Tom Shortall <tomshortall@gmail.com> Co-authored-by: Marc-Roig <marc12info@gmail.com> Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com> Co-authored-by: Pierre Noël <petersg83@users.noreply.github.com> Co-authored-by: Jorge Martin <53431308+HastaCs@users.noreply.github.com> Co-authored-by: Antti <antti.viitala@icloud.com> Co-authored-by: Shaun Brown <97027841+StrapiShaun@users.noreply.github.com> Co-authored-by: Pierre Wizla <pwizla+github@gmail.com> Co-authored-by: katekosm <71697829+katekosm@users.noreply.github.com>
What does it do?
Updates in the upload plugin Guide:
Related issue(s)/PR(s)
strapi/strapi#14398
strapi/strapi#13282