Commit graph

9411 commits

Author SHA1 Message Date
Mouse Reeve
df607a0e45
Merge pull request #3419 from bookwyrm-social/test-fix
Uses workaround for test that fails ~sometimes~
2024-08-26 12:35:32 -07:00
Mouse Reeve
bd773f41c7 Uses much simpler approach to ensuring test result order 2024-08-26 11:33:01 -07:00
Mouse Reeve
9f5ca7ae60 Uses workaround for test that fails ~sometimes~ 2024-08-25 17:53:43 -07:00
Mouse Reeve
413c26bc5e
Merge pull request #3135 from hughrun/csv
csv import and export fixes
2024-08-23 16:29:04 -07:00
Mouse Reeve
7ff1ab974e
Merge pull request #3411 from timothyjrogers/fix-link-verification-modal-unauthenticated-buttons
Narrowed is_authenticated check in verfication_modal to only restrict…
2024-08-12 19:11:07 -07:00
Tim Rogers
cf61279e18 Narrowed is_authenticated check in verfication_modal to only restrict report button 2024-08-12 19:56:26 -05:00
Mouse Reeve
cf753afaa9
Merge pull request #3410 from timothyjrogers/trim-search-whitespace
Trim search whitespace
2024-08-11 17:12:04 -07:00
Tim Rogers
f68e33ffc6 Added additional test cases for searches with leading and trailing whitespace 2024-08-11 14:02:24 -05:00
Tim Rogers
2bb77d9bf8 Updated search view to trim leading and trailing whitespace for author, book, and list query values 2024-08-11 13:01:48 -05:00
Hugh Rundle
7fc54c509c
fixes for bookwyrm csv import
- fix tests
- revert change to GenericImporter tests
- import the review name
- add extra properties to ImportItem
2024-08-10 16:37:30 +10:00
Mouse Reeve
95c2798fc7
Merge pull request #3401 from dato/comment_progress_keep_start_date
Fix reading progress `start_date` bug
2024-08-09 13:07:01 -07:00
Mouse Reeve
66dd39e6d8
Merge pull request #3151 from dato/celery_inmem
In-memory Celery backend for tests
2024-08-09 13:02:53 -07:00
Mouse Reeve
96a2fa5afc
Merge pull request #3402 from dato/ostatus_no_empty_titles
Avoid empty <title> in templates
2024-08-09 12:43:06 -07:00
Mouse Reeve
9e73ab048b
Merge pull request #3407 from bookwyrm-social/dependabot/pip/django-4.2.15
Bump django from 4.2.14 to 4.2.15
2024-08-09 12:40:24 -07:00
dependabot[bot]
b45435803a
Bump django from 4.2.14 to 4.2.15
Bumps [django](https://github.com/django/django) from 4.2.14 to 4.2.15.
- [Commits](https://github.com/django/django/compare/4.2.14...4.2.15)

---
updated-dependencies:
- dependency-name: django
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
2024-08-07 19:33:05 +00:00
Adeodato Simó
69962bb7c9 Avoid empty <title> in templates
Newer versions of tidylib complain about this.
2024-07-28 21:58:26 -03:00
Hugh Rundle
8aba8caae9
merge migration 2024-07-28 21:08:14 +10:00
Adeodato Simó
3d28019146 Use in-memory backends for Celery in tests and workflows
At the moment, this doesn't have much of an effect since most task
calls are mock'd out.
2024-07-28 06:42:11 -03:00
Adeodato Simó
f4133e0236 celerywyrm: allow broker and result backend from the environment
This allows to easily configure an in-memory transport for tests.
2024-07-28 06:42:11 -03:00
Adeodato Simó
041f2fc413 edit_readthrough: set start_date/finish_date iff present in request
Fixes: #3164
2024-07-28 05:58:38 -03:00
Adeodato Simó
812221456b Add failing test case for comment progress clobbering start_date 2024-07-28 05:56:25 -03:00
Mouse Reeve
43577f3ca0
Merge branch 'main' into csv 2024-07-27 12:26:21 -07:00
Mouse Reeve
7f773b3dbd
Merge pull request #3398 from bookwyrm-social/pylint3
Upgrade pylint to 3.x
2024-07-27 12:03:21 -07:00
Mouse Reeve
7311526f2e
Merge pull request #3375 from bookwyrm-social/pr-template
Reorganizes PR template a bit
2024-07-27 12:01:32 -07:00
Mouse Reeve
355a6071ff
Merge branch 'main' into pylint3 2024-07-27 11:48:58 -07:00
Mouse Reeve
e41b27e5f4
Merge branch 'main' into pr-template 2024-07-27 11:48:48 -07:00
Mouse Reeve
eef055159e
Merge pull request #3400 from dato/upgrade_sqlparse
Add an up-to-date sqlparse to requirements.txt
2024-07-27 11:48:38 -07:00
Adeodato Simó
08876512eb Actions: run makemigrations check with increased verbosity 2024-07-27 15:27:09 -03:00
Adeodato Simó
afd44e109c Add an up-to-date sqlparse to requirements.txt
This is used in utils/db.py to format triggers.

A migration is needed for minor whitespace fixes between
0.4.4 and 0.5.1.
2024-07-27 15:19:43 -03:00
Mouse Reeve
f0e5c334e7
Merge pull request #3399 from dato/pylint_useless_suppression_cleanup
pylint suppression cleanup
2024-07-27 09:21:17 -07:00
Mouse Reeve
300eeac0e1
Merge pull request #3374 from bookwyrm-social/landing-page-books
Use a simpler query for books to show on the landing page
2024-07-27 08:47:02 -07:00
Mouse Reeve
dc54f28e17 Type fix 2024-07-27 08:44:44 -07:00
Mouse Reeve
8ce29849a9
Merge pull request #3393 from bookwyrm-social/dependabot/pip/django-4.2.14
Bump django from 4.2.11 to 4.2.14
2024-07-27 08:36:17 -07:00
Adeodato Simó
0b87aacfce pylint: enable useless-suppression lint and perform cleanup 2024-07-27 03:47:35 -03:00
Adeodato Simó
81ee5e945f pylint: remove unused global suppressions 2024-07-27 03:42:27 -03:00
Adeodato Simó
1a0fbac76c pylint: upgrade to 3.2.6
This only requires fixing:

    E0606: Possibly using variable 'results' before assignment
    E0606: Possibly using variable 'input_type' before assignment
2024-07-27 03:38:47 -03:00
Adeodato Simó
2cdbddca09 .pylintrc: use symbolic names for message suppressions 2024-07-27 03:10:47 -03:00
Hugh Rundle
1608ca6401
improve formatting 2024-07-27 13:16:34 +10:00
Hugh Rundle
93c6b76dab
Merge branch 'main' into csv 2024-07-27 13:04:38 +10:00
Hugh Rundle
94dfbbcc05
fix BookwyrmBooksImporter and tests
- change class attribute to instance attribute for mappings
- remove comment from test
- order import retry jobs in generic importer test

This last change seems innocuous but I may be missing something more fundamental  - it was otherwise failing when multiple tests are run, I think because running tests in parallel led to import jobs getting out of order?
2024-07-27 12:55:15 +10:00
Bart Schuurmans
ab307388f4
Merge pull request #3384 from lo48576/fix/css-path-prefix-with-s3-backend
Fix CSS path prefix when S3 storage is used
2024-07-14 13:24:02 +02:00
dependabot[bot]
8a235bcda1
Bump django from 4.2.11 to 4.2.14
Bumps [django](https://github.com/django/django) from 4.2.11 to 4.2.14.
- [Commits](https://github.com/django/django/compare/4.2.11...4.2.14)

---
updated-dependencies:
- dependency-name: django
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
2024-07-10 22:16:44 +00:00
Hugh Rundle
06d6360082
merge latest changes and add tests 2024-06-30 18:54:59 +10:00
Hugh Rundle
e5b260e3ee
Merge branch 'main' into csv 2024-06-29 16:03:15 +10:00
Mouse Reeve
3236003107
Merge pull request #3378 from hughrun/get-books-for-user
possible fix for #3372 - user export timeouts
2024-06-22 20:24:36 -07:00
YOSHIOKA Takuma
1a2f434514
Fix CSS path prefix when S3 storage is used
django-sass-processor 1.4 looks up OPTIONS using `sass_processor`
instead of `staticfiles`.

Fixes #3383.
2024-06-13 05:03:10 +09:00
Hugh Rundle
1d4119e853
LOL
remove Q import so pylint doesn't grumble
2024-06-09 10:59:11 +10:00
Hugh Rundle
261e794c1c
possible fix for #3372 - user export timeouts
This definitely needs to be tested on a large DB but I believe it may fix the timeouts b.s. gets when running user exports.

Instead of a gigantic single DB query with heaps of joins, we instead just do a series of simple queries and then use union()
to pull them into a de-duped queryset.

If I understand the results from explain() correctly, this is a massive reduction in DB work:

Unique  (cost=195899.15..198201.71 rows=11808 width=19220)

vs

Unique  (cost=150.28..153.44 rows=16 width=19220)
2024-06-09 10:34:22 +10:00
Mouse Reeve
2e675474a9 Reorganizes PR template a bit
I found the template a little overwhelming, so this is an attempt to
make it a little more navigable and slightly less effortful.
2024-06-08 08:31:34 -07:00
Mouse Reeve
3f08d6d8c4 Use a simpler query for books to show on the landing page 2024-06-08 08:18:54 -07:00