search

wagtail / wagtail-localize

Translation plugin for Wagtail CMS
https://wagtail-localize.org/
Other
222 stars 84 forks source link

Move revision publishing in `TranslationSource` to `transaction.on_commit` #720

Closed zerolab closed 11 months ago

zerolab commented 11 months ago

Replaces #711 to bring up to date with main

codecov-commenter commented 11 months ago

Codecov Report

All modified lines are covered by tests :white_check_mark:

Comparison is base (7e75cbd) 93.24% compared to head (4b16329) 93.29%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #720 +/- ## ========================================== + Coverage 93.24% 93.29% +0.05% ========================================== Files 47 47 Lines 3921 3921 Branches 583 583 ========================================== + Hits 3656 3658 +2 + Misses 154 153 -1 + Partials 111 110 -1 ``` | [Files](https://app.codecov.io/gh/wagtail/wagtail-localize/pull/720?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=wagtail) | Coverage Δ | | |---|---|---| | [wagtail\_localize/models.py](https://app.codecov.io/gh/wagtail/wagtail-localize/pull/720?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=wagtail#diff-d2FndGFpbF9sb2NhbGl6ZS9tb2RlbHMucHk=) | `96.23% <100.00%> (ø)` | | ... and [1 file with indirect coverage changes](https://app.codecov.io/gh/wagtail/wagtail-localize/pull/720/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=wagtail)

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