-
Notifications
You must be signed in to change notification settings - Fork 926
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add wagtail-localize-smartling to the project, for Smartling L10N support #14794
Conversation
Review tagging is more about awareness than close reviewing right now. Good to get this merged (when the infra is ready) |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #14794 +/- ##
==========================================
+ Coverage 77.29% 77.38% +0.09%
==========================================
Files 160 161 +1
Lines 8281 8306 +25
==========================================
+ Hits 6401 6428 +27
+ Misses 1880 1878 -2 ☔ View full report in Codecov by Sentry. |
887fe4d
to
7931a8b
Compare
Secrets deployed to dev and will not crash when code reaches Stage or Prod because we have non-exploding (but deliberately invalid) defaults set of |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
r+wc
everything other than the question about env vars looks good.
|
||
.. note:: | ||
|
||
It's worth investing 15 mins in watching the `Wagtail Localize original demo`_ to get a good feel of how it can work. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just noticing the above text looks like it's wrapped whereas this new text isn't.
c0dc2ab
to
6962f4d
Compare
…can monitor it with a DMS
ed6edbb
to
51254b6
Compare
One-line summary
This changeset drops in wagtail-localize-smartling, which will flow strings from Wagtail to Smartling for L10N and then back again.
Significant changes and points to review
0.2.3 is an early-ish, Minimum Lovable Product version, and there may well be some fast-follow changes from the vendor - those are in discussion at the mo, and we can pick up anything else we want, as it's "our" codebase now.
I've tested it manually, locally, for now, with sandbox-y API keys and it works for the core cases. There are some rough edges but we'll sort them out.
The infra PR needed to accompany these changes will need to:
Issue / Bugzilla link
Resolves #14774
Testing
N/A for now - I'll record a video demo, but waiting to see if there'll be a 0.2.4 soon (or not yet)