Skip to content

Fix thumnail : don't test if media exists#659

Merged
amandine-sahl merged 2 commits intodevelopfrom
fix/thumnail_generation
Feb 16, 2026
Merged

Fix thumnail : don't test if media exists#659
amandine-sahl merged 2 commits intodevelopfrom
fix/thumnail_generation

Conversation

@amandine-sahl
Copy link
Contributor

@amandine-sahl amandine-sahl commented Dec 18, 2025

@amandine-sahl amandine-sahl force-pushed the fix/thumnail_generation branch from 04fd120 to 3c5f927 Compare December 18, 2025 15:54
@noelmartinon
Copy link

Si le media .png n'est pas trouvé, ce sera pas mal ensuite de vérifier si le même en .jpg existe avant de continuer. Dans mon répertoire thumb, j'ai parfois ce cas là où je n'ai que le "500x500.jpg" par exemple.

@camillemonchicourt
Copy link
Member

Ah OK, je croyais que toutes les vignettes étaient au format PNG, et que justement on voulait basculer désormais en JPG (#644).
Ça serait bien d'avoir qu'un seul format à terme et plutôt JPG si possible.

@codecov
Copy link

codecov bot commented Dec 19, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 53.92%. Comparing base (6e5d283) to head (fa7017a).
⚠️ Report is 1 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #659      +/-   ##
===========================================
- Coverage    53.96%   53.92%   -0.04%     
===========================================
  Files           39       39              
  Lines         2763     2763              
===========================================
- Hits          1491     1490       -1     
- Misses        1272     1273       +1     
Flag Coverage Δ
pytest 53.92% <100.00%> (-0.04%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@amandine-sahl amandine-sahl force-pushed the fix/thumnail_generation branch 2 times, most recently from cb8c07f to 4a3dc74 Compare December 19, 2025 10:36
@jpm-cbna
Copy link
Contributor

Ah OK, je croyais que toutes les vignettes étaient au format PNG, et que justement on voulait basculer désormais en JPG (#644). Ça serait bien d'avoir qu'un seul format à terme et plutôt JPG si possible.

Oui, JPG est à préférer vis à vis de PNG pour éviter d'occuper trop d'espace de stockage.

Il y a eu aussi un changement du format du web service. Cela a cassé la rétro-compatibilité avec les anciens "thumbnail".
Or, avec l'API de l'INPN non fonctionnelle, cela empêche de récupérer les miniatures. Dans le ticket #644, j'ai indiqué comment restaurer l'ancien format du web service pour que les miniatures, générées avant la modification du web service, soient récupérées correctement.

@amandine-sahl amandine-sahl added this to the 2.2.4 milestone Feb 5, 2026
@amandine-sahl amandine-sahl force-pushed the fix/thumnail_generation branch from 47b915c to c9f005e Compare February 5, 2026 15:54
lint

Run test on ci

bug test
@amandine-sahl amandine-sahl force-pushed the fix/thumnail_generation branch from d8d452a to fa7017a Compare February 6, 2026 13:31
@amandine-sahl amandine-sahl merged commit 1928870 into develop Feb 16, 2026
7 checks passed
@amandine-sahl amandine-sahl deleted the fix/thumnail_generation branch February 16, 2026 11:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Comments