Closed stevejalim closed 6 days ago
Attention: Patch coverage is 90.47619%
with 2 lines
in your changes missing coverage. Please review.
Project coverage is 78.81%. Comparing base (
9d4487e
) to head (4244edd
). Report is 3 commits behind head on main.
Files with missing lines | Patch % | Lines |
---|---|---|
...edrock/cms/wagtail_localize_smartling/callbacks.py | 90.47% | 2 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
🚨 Try these New Features:
One-line summary
This changset adds support for sending a visual context (a HTML document + relevant URL) to Smartling so that the CAT "live translation preview" tool can be used easily.
Significant changes and points to review
Adds a callback for wagtail-localize-smartling to send visual context to Smartling
Updates our
wagtail-localize-smartling
andwagtaildraftsharing
dependencies, to enable the callback mentioned above to be used (and also to generate a useful URL with it)Issue / Bugzilla link
Resolves #14938
Testing
I've tested this with @cocomo over a Zoom call, so code-level review here is enough