-
-
Notifications
You must be signed in to change notification settings - Fork 820
Enable i18n plugin, configure, and prepare for switch to Weblate #637
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
Conversation
0519050 to
4a86e9d
Compare
6aa111c to
4a86e9d
Compare
|
Visit the preview URL for this PR (updated for commit 014343a): https://beeware-org--pr637-switch-to-weblate-68en8zz8.web.app (expires Thu, 05 Jun 2025 23:47:53 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: b0da44bc067e7d9a4255c77cb2c5fce572218cec |
|
I've run the .po files through DeepL to generate initial translations; however, the preview site isn't showing them. It's definitely hitting the plugin (I needed to do some poking to get gettext into the build environment)... but the .po content isn't being rendered on the .lr pages. |
|
Another thing that just occurred to me - there's a bunch of code on the site (driven by the beeware plugin) that identifies if a translated page is out of date. That code is no longer needed (or, at least, needs to be substantially reworked to be accurate). I think we might be able to drop the beeware plugin entirely... |
So.... remember that
So! It looks like the current preview sequence is I think, ultimately, the publish sequence may need to be updated as well. However, we can cross that bridge when we get to it. It's certainly going to add some time to the build and preview processes, but if this is how it needs to happen, I suppose we deal with it. It's not something that's happening every few minutes or anything, typically, so I suppose it's fine? |
Unless this is causing issues with the current translation setup (I have no idea if it's part of the problem), I think this should be filed as an issue and bumped to a future fix. |
|
For posterity: It turns out the double |
freakboy3742
left a comment
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.
(sweating intensifies)

Prepares website for switch to Weblate for translations. Includes the following changes:
.poand.pottranslation files added.BeeWare.lektorprojectto include Lektor management of installing i18n plugin..gitignoreto ignore generated language files.Refs #634
PR Checklist: