-
Notifications
You must be signed in to change notification settings - Fork 0
FOLLOW-UP: Make MaxFileSize configurable via generator flag #271
Copy link
Copy link
Open
Labels
follow-upFollow-up task from PR reviewFollow-up task from PR reviewfrom-reviewIssue originated from PR reviewIssue originated from PR review
Description
Context
This follow-up task was identified during the review of PR #267.
Source PR: #267
PR Title: feat: add file upload & storage support (Roadmap 2.2)
Suggested by: @claude[bot]
Task Description
The MaxFileSize in the generated WithUpload config is hardcoded to 10 << 20 (10 MB). This will be wrong for many use cases (e.g., video uploads need more, avatar thumbnails need less).
Options:
- Add a
--max-file-sizeflag tolvt gen resource - Make it a named constant at the top of the generated handler
- Allow per-field size limits in the field spec (e.g.,
photo:image:5mb)
The simplest approach is a generator flag that sets the value for all file fields in that resource.
This issue was automatically created by prmonitor from PR review comments.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
follow-upFollow-up task from PR reviewFollow-up task from PR reviewfrom-reviewIssue originated from PR reviewIssue originated from PR review