#28184 closed New feature (fixed)
FileField storage param should allow a callable
Reported by: | miigotu | Owned by: | miigotu |
---|---|---|---|
Component: | File uploads/storage | Version: | dev |
Severity: | Normal | Keywords: | callable storage filefield |
Cc: | Triage Stage: | Ready for checkin | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
For example, when using S3 I may have a field that I want to use private acl, while other fields are not using private storage. Locally or in different deployments I may not be using S3, and the storage is different.
Change History (12)
comment:1 Changed 6 years ago by
Easy pickings: | unset |
---|---|
Patch needs improvement: | set |
Summary: | FileField storages param should allow a callable → FileField storage param should allow a callable |
Triage Stage: | Unreviewed → Accepted |
comment:2 Changed 6 years ago by
Patch needs improvement: | unset |
---|
Improved patch, passes all tests now.
comment:3 Changed 5 years ago by
Needs documentation: | set |
---|---|
Patch needs improvement: | set |
comment:5 Changed 5 years ago by
comment:6 Changed 5 years ago by
Needs documentation: | unset |
---|---|
Patch needs improvement: | unset |
comment:7 Changed 5 years ago by
Version: | 1.11 → master |
---|
comment:8 Changed 5 years ago by
Patch needs improvement: | set |
---|
comment:9 Changed 3 years ago by
Patch needs improvement: | unset |
---|
comment:10 Changed 3 years ago by
Triage Stage: | Accepted → Ready for checkin |
---|
Note: See
TracTickets for help on using
tickets.
Tentatively accepting. This idea also came up in #24648. The PR has test failures and documentation changes are also required. See the PatchReviewChecklist.