Skip to content

Conversation

@raydeal
Copy link
Contributor

@raydeal raydeal commented Oct 28, 2025

Fixes #1976

@coveralls
Copy link

coveralls commented Oct 28, 2025

Coverage Status

coverage: 96.447% (+0.001%) from 96.446%
when pulling 1076801 on raydeal:fix-1976
into dedac4c on rafalp:main.

@rafalp rafalp assigned rafalp and raydeal and unassigned rafalp Oct 30, 2025
@rafalp rafalp added the area: backend This issue involves Python, Django or dependency (eg. database) label Oct 30, 2025
@rafalp rafalp added this to the Misago 0.40 alpha 1 milestone Oct 30, 2025
Copy link
Owner

@rafalp rafalp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM but one nit.

attachment_thumbnail_width=150,
attachment_thumbnail_height=100,
)
def test_store_uploaded_file_stores_image_file_scaled_down_with_thumbnail(
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please move this test under test_store_uploaded_file_stores_image_file_scaled_down so they are together

@rafalp rafalp merged commit 7b8e29c into rafalp:main Nov 3, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: backend This issue involves Python, Django or dependency (eg. database)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Max attachment image size breaks thumbnails

3 participants