-
Notifications
You must be signed in to change notification settings - Fork 1
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
Docs: Add Info about file size in FileUploader
#DS-993
#1822
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for spirit-design-system-storybook ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for spirit-design-system ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
0f58a14
to
be3f93f
Compare
packages/web-twig/src/Resources/components/FileUploader/README.md
Outdated
Show resolved
Hide resolved
685a5d8
to
718213f
Compare
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.
Nicely documented. I love it. Thanks :-)
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.
Astonishing and valuable. I learned something new today, thank you!
module.exports = { | ||
experimental: { | ||
serverActions: { | ||
bodySizeLimit: '1mb', |
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.
Is the unit really all lowercase? I'd consider dropping a comment on it because the case is important for IT size units.
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.
This is copied right from the docs: https://nextjs.org/docs/app/api-reference/config/next-config-js/serverActions#bodysizelimit
🤷
And that is why I asked @pavelklibani to make it more accurate and with examples. Because this is happening
🤷
Description
The primary update is in the React README, which is now referenced by other packages.
base-10
andbase-2
systems.PHP
andNext.js
configurations, along with their shorthands.Additional context
Issue reference
FileUploader: file size limit dokumentace