Closed
Bug 1089375
Opened 10 years ago
Closed 10 years ago
[Compat Data] Spin out drf_cached_reads as separate Django app
Categories
(developer.mozilla.org Graveyard :: General, enhancement)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jwhitlock, Unassigned)
References
Details
(Whiteboard: [specification][type:feature])
What problems would this solve?
===============================
drf_cached_reads could be a generally useful project that would grow its own contributor base, attracting improvements without minimal developer resources.
Who would use this?
===================
Django REST Framework developers who have APIs with a strong read-only use case
What would users see?
=====================
A project with it's own github repo, published on PyPi and pip installable
What would users do? What would happen as a result?
===================================================
Users would integrate the code into their own project, file bugs, and contribute patches. Useful code would be integrated back into the compact data project.
Is there anything else we should know?
======================================
drf_cached_reads is already structured as a standalone Django app. It may need a rename (I'd prefer 'drf_instance_cache'), and some additional tests.
Comment 1•10 years ago
|
||
Commits pushed to master at https://github.com/mozilla/web-platform-compat
https://github.com/mozilla/web-platform-compat/commit/bc9e38c2f5c6b82b1f8cf5cdde895c4585afbad4
bug 1089375 - Switch to drf_cached_instances
drf_cached_reads moved to separate repo:
https://github.com/jwhitlock/drf-cached-instances
https://github.com/mozilla/web-platform-compat/commit/25ac58f91a387e5aa13038b0e0df1cc2f1b0253e
fix bug 1089375 - Refactor duplicated cache code
Move code for loading primary keys of related items into new function,
call from the loader and serializers.
https://github.com/mozilla/web-platform-compat/commit/ef132cc31180000a488a1fedbdad5d71938005cd
Merge pull request #11 from jwhitlock/1089375_spin_out_drf_cached_reads
bug 1089375 - spin out drf_cached_reads
Updated•10 years ago
|
Status: UNCONFIRMED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 2•10 years ago
|
||
Commit pushed to master at https://github.com/mozilla/web-platform-compat
https://github.com/mozilla/web-platform-compat/commit/4b2ae7d555790f56ea75e41d7abd3a89170dad43
bug 1089375 - setting is USE_DRF_INSTANCE_CACHE
When the code was moved to drf_cached_instances, the setting to disable
the cache changed from USE_INSTANCE_CACHE to USE_DRF_INSTANCE_CACHE.
This commit changes the setting in the project and the documentation.
Updated•5 years ago
|
Product: developer.mozilla.org → developer.mozilla.org Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•