Use a simpler query for books to show on the landing page - Githubissues

bookwyrm-social / bookwyrm

Social reading and reviewing, decentralized with ActivityPub

http://joinbookwyrm.com/

Other

2.17k stars 255 forks source link

Use a simpler query for books to show on the landing page #3374

Open mouse-reeve opened 3 weeks ago

mouse-reeve commented 3 weeks ago

Are you finished?

Linters

[ ] I have checked my code with black, pylint, and mypy, or ./bw-dev formatters

Tests

[ ] My changes do not need new tests
[ ] All tests I have added are passing
[ ] I have written tests but need help to make them pass
[ ] I have not written tests and need help to write them

What type of Pull Request is this?

[ ] Bug Fix
[ ] Enhancement
[ ] Plumbing / Internals / Dependencies
[x] Refactor

Does this PR change settings or dependencies, or break something?

[ ] This PR changes or adds default settings, configuration, or .env values
[ ] This PR changes or adds dependencies
[ ] This PR introduces other breaking changes

Description

I'm going through time consuming SQL queries and this one was doing too much. I think recently updated books is perfectly adequate in lieu of recently reviewed books.

Documentation

[ ] New or amended documentation will be required if this PR is merged
[ ] I have created a matching pull request in the Documentation repository
[ ] I intend to create a matching pull request in the Documentation repository after this PR is merged