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)

All
Other
enhancement
Not set
normal

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.
Blocks: 996570
Severity: normal → enhancement
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
Status: UNCONFIRMED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
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.
Product: developer.mozilla.org → developer.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.