#18929: CachedFilesMixin is not compatible with S3BotoStorage
-------------------------------------+------------------------------------
Reporter: idanzalz@… | Owner: nobody
Type: Bug | Status: new
Component: contrib.staticfiles | Version: 1.4
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 1 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
-------------------------------------+------------------------------------
Changes (by aaugustin):
* needs_better_patch: 0 => 1
* needs_tests: 0 => 1
* stage: Unreviewed => Accepted
Comment:
This ticket looks valid. But the current patch isn't acceptable because it
breaks tests (as mentioned by Karen on the pull request).
To move forward, the code should make it clearer which kind of path
(filesystem, URL-unescaped, URL-escaped) in accepted and return by storage
functions, and more tests must be added.
--
Ticket URL: <https://code.djangoproject.com/ticket/18929#comment:3>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit https://groups.google.com/groups/opt_out.
