Commit graph

7249 commits

Author SHA1 Message Date
Mouse Reeve
5bb001e15f Show avatars in user admin view
I find this helpful as a rough hueristic for how engaged a user is -- if
they've never added an avatar, it's often the case that they never
really got further in to using the app than just registering a username.
Also, on one occassion, a user had an avatar that violated the instance
ToS, which would have been handy to see at a glance in this list.
2022-07-08 09:10:29 -07:00
Mouse Reeve
16a76dba72
Merge pull request #2197 from bookwyrm-social/admin-perms
Admin perms
2022-07-07 14:15:38 -07:00
Mouse Reeve
a972c58f3b Fixes errors in permissions 2022-07-07 14:01:51 -07:00
Mouse Reeve
948f2964ac Updates admin tests to use perms instead of superuser
Superuser overrides all perms, so this was hiding simple typo-related
bugs in the permissions system
2022-07-07 14:01:05 -07:00
Mouse Reeve
2eaffc7249
Merge pull request #2195 from bookwyrm-social/a-farcical-parade-of-errors-when-reading-multiple-editions
A farcical parade of errors when reading multiple editions
2022-07-07 12:31:56 -07:00
Mouse Reeve
742d97b177 Python formatting 2022-07-07 12:23:10 -07:00
Mouse Reeve
76c466ee45 Fixes cache invalidation for editions
Your reading status is shown across all editions of a work, so when you
change your status in relation to the edition you're currently reading,
it needs to invalidate the cached values for all editions of that work.
2022-07-07 12:21:24 -07:00
Mouse Reeve
076ba6f0ae Dramatically reduce cache duration for reading statuses
I originally set this for a very long timeout because this value should
be invalidated when it needs to be by the models, and if that worked
perfectly, this would reduce queries dramatically for books that show up
in ones feed frequently, but don't change status (for example, a book
you read and your friend is currently posting about). In practice, of
course, there are errors in invalidating this cache which leave this
value appearing extremely broken and it's next to impossible to fix.

This change makes each of the timeouts related to reading an hour, which
will still give performance benefit when browsing the site (especially
for loading the same book multiple times on a page), but resolve
naturally if the cache gets into a bad state.
2022-07-07 12:06:06 -07:00
Mouse Reeve
8e18c21460
Merge pull request #2192 from bookwyrm-social/tests
Tests for image generator file
2022-07-07 11:47:18 -07:00
Mouse Reeve
1a082b5ee8 Settings 2022-07-07 11:37:34 -07:00
Mouse Reeve
3d2752e388
Merge pull request #2193 from bookwyrm-social/locales
Updates locales
2022-07-07 11:04:18 -07:00
Mouse Reeve
ae58d2f74a Updates locales
Hella new translation strings
2022-07-07 10:48:28 -07:00
Mouse Reeve
634ed50044 Tests for image generator file
This isn't what I need to be working on right now but it bugs me
immensely that the test coverage is at 89% and not 90% and this was the
lowest hanging fruit to get more coverage. Truly a me problem.
2022-07-07 10:24:19 -07:00
Mouse Reeve
0b7c8e8dc0
Merge pull request #2082 from bookwyrm-social/notifications
Notifications refactor
2022-07-07 09:34:19 -07:00
Mouse Reeve
a5e761b134 Group accept and leave notification 2022-07-07 08:55:17 -07:00
Mouse Reeve
e032d7fd5a
Merge pull request #2190 from bookwyrm-social/forgot-password-email
Update password reset copy so as not to reveal whether the email exists
2022-07-06 20:00:24 -07:00
Mouse Reeve
fd5e513ad6 Update password reset copy so as not to reveal whether the email exists
A malicious user could use this to test which email addresses are in the
user database.
2022-07-06 19:34:00 -07:00
Mouse Reeve
cded3e973d Fixes html on notifications page 2022-07-06 19:16:14 -07:00
Mouse Reeve
26a1f75e91 Group notification types by feature 2022-07-06 19:15:48 -07:00
Mouse Reeve
6a5323c6e7 More tests for more notification types 2022-07-06 19:15:35 -07:00
Mouse Reeve
96bf99034c
Merge pull request #2189 from bookwyrm-social/extraneous_page_number
Removed incorrect and duplicate page indicator from content status
2022-07-06 16:38:09 -07:00
Mouse Reeve
060afd599d
Merge pull request #2188 from bookwyrm-social/more-tests
More tests
2022-07-06 15:31:41 -07:00
Mouse Reeve
4bb3a7e4cd Removed incorrect and duplicate page indicator from content status 2022-07-06 15:27:48 -07:00
Mouse Reeve
04f9b9180d Adds test file for utils 2022-07-06 15:07:46 -07:00
Mouse Reeve
d3023f350d Adds tests to site admin view 2022-07-06 15:04:27 -07:00
Mouse Reeve
221b5138e8 Adds merge migration 2022-07-06 14:42:58 -07:00
Mouse Reeve
5a6ef7ccd8 Merge branch 'main' into notifications 2022-07-06 14:41:15 -07:00
Mouse Reeve
dda6126329 Fixes typo in group notification 2022-07-06 14:33:45 -07:00
Mouse Reeve
2cda9d5567 Test leaving a group (as opposed to being removed)
This test will catch my typo in generating the notifications
2022-07-06 14:31:38 -07:00
Mouse Reeve
d09aa2ded2
Merge pull request #2187 from bookwyrm-social/missing-migration
Adds another missing migration
2022-07-06 12:33:07 -07:00
Mouse Reeve
079f6d304e Python formatting 2022-07-06 12:22:07 -07:00
Mouse Reeve
9fceeb3eda Adds another missing migration 2022-07-06 12:17:24 -07:00
Mouse Reeve
476a7b73a2
Merge pull request #2186 from bookwyrm-social/test-coverage
Adds test coverage command and some more miscellaneous tests
2022-07-06 11:32:49 -07:00
Mouse Reeve
b2a8a04092 Removes unnecessary imports 2022-07-06 11:24:04 -07:00
Mouse Reeve
9c21d1c06f Adds more automod tests 2022-07-06 11:15:30 -07:00
Mouse Reeve
7a9d320afd Expands automod view test 2022-07-06 10:52:50 -07:00
Mouse Reeve
574c1db732 Adds more templatetag tests 2022-07-06 10:52:45 -07:00
Mouse Reeve
87434fbb9d Adds a couple more edit book views tests 2022-07-06 10:38:22 -07:00
Mouse Reeve
f5638c1e44 Removed unused helper function 2022-07-06 10:05:55 -07:00
Mouse Reeve
f4dd14acad Adds more webfinger tests for helper functions 2022-07-06 10:05:47 -07:00
Mouse Reeve
c48fbd8753 Separates reporting and report admins tests 2022-07-06 09:47:48 -07:00
Mouse Reeve
afd3b27d78 Adds coverage command to bw-dev 2022-07-06 09:47:33 -07:00
Mouse Reeve
b61a4ab994 Adds tests for unnotify 2022-07-06 08:51:35 -07:00
Mouse Reeve
e8a570b687
Merge pull request #2185 from bookwyrm-social/userless-link-domain
Adds missing migration
2022-07-05 18:09:45 -07:00
Mouse Reeve
003ef65db5 Adds missing migration 2022-07-05 18:01:10 -07:00
Mouse Reeve
cccf0ec3a0
Merge pull request #2184 from bookwyrm-social/userless-link-domain
Handle link domains with no associated user
2022-07-05 17:51:12 -07:00
Mouse Reeve
d4f8a5f448 Fixes links to report modals 2022-07-05 17:43:03 -07:00
Mouse Reeve
4f386e8c4e Allow reports with no associated user 2022-07-05 17:19:03 -07:00
Mouse Reeve
0be4fec167 Don't assume that link domains have a known user
This is an edge case, and they OUGHT to have a user, but if the user was
deleted for a remote link, the field won't be able to resolve and the
user will be unset.
2022-07-05 16:51:03 -07:00
Mouse Reeve
24349b0a4c Updates notification view tests 2022-07-05 16:20:27 -07:00