Skip to content

Use Django caching - #809

Merged
geekygirlsarah merged 5 commits into
codethesaurus:mainfrom
geekygirlsarah:main
Oct 5, 2026
Merged

geekygirlsarah merged 5 commits into
codethesaurus:mainfrom
geekygirlsarah:main

Conversation

@geekygirlsarah

@geekygirlsarah geekygirlsarah commented Oct 5, 2026 •

Copy link
Copy Markdown
Member

What GitHub issue does this PR apply to?

Issue
#487

PR
#646

What changed and why?

Since @LittleBigProgramming 's original PR to do caching, I unfortunately hadn't really tested it and time passed and the code base strayed from those commits. But I wanted to still implement it, and use @LittleBigProgramming 's original code if possible.

This pulls in their changes, then fixes things that evolved, notably that Django 4.x is now Django 6.x. The original intent is still there, just tried to resolve any issues that came up after.

(It does mean I'll have to close #646 though, sorry!)

It also adds some extra security, linting, and other checks into the CI process, and likely will break upon merging.

AI bots used in the process of making this PR

OpenCode's Big Pickle model
Review and some manual work by myself

Checklist

  • I claimed any associated issue(s) and they are not someone else's
  • I have looked at documentation to ensure I made any revisions correctly
  • I tested my changes locally to ensure they work
  • For language files, I have validated the edited files are valid JSON and data shows up correctly
  • For website code edits, I have added or edited any appropriate unit tests for my changes
  • If I used artificial intelligence (AI), I have reviewed the suggested changes and ensured they work correctly

Any additional comments or things to be aware of while reviewing?

An extra thank you again to @LittleBigProgramming , who's contributed a lot to this project over the years!

Sarah Withee and others added 4 commits September 18, 2026 12:16
development environments which will use Dummy Caching
(https://docs.djangoproject.com/en/4.1/topics/cache/#dummy-caching-for-development)

Site wide cache will stored in cache_table, to run this initially the
command of `python manage.py createcachetable` will need to be run after
the deployment of this feature.
Comment thread web/models.py Fixed
Comment thread web/models.py Fixed
@geekygirlsarah
geekygirlsarah merged commit 6f99da3 into codethesaurus:main Oct 5, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants