Add test coverage and improve localization - #3
Merged
Conversation
…xed openfoodfacts importer to import actual sugar values
…ng German locale translations
- Keep OIDC-created users non-staff while preserving existing permissions on update - Fix admin importer route ordering and i18n context availability - Normalize German characters for ASCII URL slugs and remove duplicate ingredient slug field - Make tag seeding non-destructive and reject animated recipe image uploads - Reconcile migration state and add regression coverage
Welcome to Codecov 🎉Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests. ℹ️ You can also turn on project coverage checks and project coverage reporting on Pull Request comment Thanks for integrating Codecov - We've got you covered ☂️ |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🚀 Summary
This PR promotes the current development branch to master. It includes localization fixes, upload security hardening, importer hardening, test coverage improvements, and several release-readiness fixes.
🧩 Related Issues
N/A
🔍 What Changed?
Improved localization:
ä,ö,ü, andßin non-URL textHardened recipe image handling:
Hardened recipe importers:
Improved security/configuration:
SECRET_KEYfail loudly instead of silently generating a random key.envsetup expectationsFixed project readiness issues:
Improved nutrition/import behavior:
Improved testing and coverage:
Other cleanup:
🧪 How to Test
Manual checks worth doing before merge:
✅ Checklist
📎 Additional Notes
This PR includes the accumulated development changes since the last master release and is intended as the next development-to-master promotion.