thunderstore-io / Thunderstore

Thunderstore is a mod database and API for downloading mods. Thunderstore Discord: https://discord.thunderstore.io/
https://thunderstore.io/
GNU Affero General Public License v3.0
138 stars 28 forks source link

Add CommunityAggregatedFields #886

Closed anttimaki closed 1 year ago

anttimaki commented 1 year ago

Current dependencies on/for this PR:

This comment was auto-generated by Graphite.

anttimaki commented 1 year ago

TODO: ensure the fields are updated in an efficient manner, add tests, doc strings, celery job etc.

codecov[bot] commented 1 year ago

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (4a901b7) 92.21% compared to head (b086bca) 92.23%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #886 +/- ## ========================================== + Coverage 92.21% 92.23% +0.02% ========================================== Files 270 271 +1 Lines 7741 7778 +37 Branches 737 739 +2 ========================================== + Hits 7138 7174 +36 - Misses 501 502 +1 Partials 102 102 ``` | [Files](https://app.codecov.io/gh/thunderstore-io/Thunderstore/pull/886?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thunderstore-io) | Coverage Δ | | |---|---|---| | [django/conftest.py](https://app.codecov.io/gh/thunderstore-io/Thunderstore/pull/886?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thunderstore-io#diff-ZGphbmdvL2NvbmZ0ZXN0LnB5) | `97.12% <100.00%> (+0.01%)` | :arrow_up: | | [...understore/api/cyberstorm/serializers/community.py](https://app.codecov.io/gh/thunderstore-io/Thunderstore/pull/886?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thunderstore-io#diff-ZGphbmdvL3RodW5kZXJzdG9yZS9hcGkvY3liZXJzdG9ybS9zZXJpYWxpemVycy9jb21tdW5pdHkucHk=) | `100.00% <100.00%> (ø)` | | | [...jango/thunderstore/community/context\_processors.py](https://app.codecov.io/gh/thunderstore-io/Thunderstore/pull/886?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thunderstore-io#diff-ZGphbmdvL3RodW5kZXJzdG9yZS9jb21tdW5pdHkvY29udGV4dF9wcm9jZXNzb3JzLnB5) | `73.68% <ø> (-1.32%)` | :arrow_down: | | [django/thunderstore/community/tasks.py](https://app.codecov.io/gh/thunderstore-io/Thunderstore/pull/886?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thunderstore-io#diff-ZGphbmdvL3RodW5kZXJzdG9yZS9jb21tdW5pdHkvdGFza3MucHk=) | `100.00% <100.00%> (ø)` | | | [...store/frontend/api/experimental/views/frontpage.py](https://app.codecov.io/gh/thunderstore-io/Thunderstore/pull/886?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thunderstore-io#diff-ZGphbmdvL3RodW5kZXJzdG9yZS9mcm9udGVuZC9hcGkvZXhwZXJpbWVudGFsL3ZpZXdzL2Zyb250cGFnZS5weQ==) | `100.00% <100.00%> (ø)` | | | [django/thunderstore/community/models/community.py](https://app.codecov.io/gh/thunderstore-io/Thunderstore/pull/886?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thunderstore-io#diff-ZGphbmdvL3RodW5kZXJzdG9yZS9jb21tdW5pdHkvbW9kZWxzL2NvbW11bml0eS5weQ==) | `93.23% <97.36%> (+0.37%)` | :arrow_up: |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

anttimaki commented 1 year ago

@MythicManiac ready for review. Since last you checked I made the batch_size optional, added babby's first celery task in a true monkey-see-monkey-do manner, and updated the linting warning commit.