EnxDev opened a new pull request, #40764:
URL: https://github.com/apache/superset/pull/40764
### SUMMARY
The translation regression CI check flags that several source strings on
`master` were added or reworded without `superset/translations/messages.pot`
being re-extracted, leaving the extracted catalog out of sync with the source.
This PR re-runs the standard translation sync and resolves the resulting
fuzzy entries.
What this does:
- Re-extracts translatable strings via
`./scripts/translations/babel_update.sh`,
regenerating `messages.pot` and merging into every `.po` file.
- For each entry that babel re-fuzzed from a reworded source `msgid`,
removes the
`#, fuzzy` flag and clears the `msgstr` to `""`, so native translators can
re-fill them via Crowdin instead of shipping a mismatched borrowed
translation.
Affected `msgid`s (all originate from features already merged to `master` —
ag-grid numbered column, Google Sheets connector, and DB engine error
handling):
- `Add numbered column`
- `Whether to display the numbered column`
- `Custom SQL fields cannot contain set operations.`
- `Not found`
- `Unexpected HTTP 401 response. Check your credentials.`
Resolved across all 25 affected languages (the regression check reported 18;
babel
also re-fuzzed these strings in es, fa, pl, pt_BR, tr, zh, zh_TW, which are
fixed here too).
Note: `messages.pot`'s `Generated-By` reverts from `Babel 2.18.0` to `Babel
2.17.0`.
2.17.0 is the version pinned in `requirements/*.txt` and
`superset/translations/requirements.txt`; the committed `.pot` had been
generated
with an off-pin newer Babel.
### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
- Before
<img width="932" height="943" alt="before"
src="https://github.com/user-attachments/assets/f28ec439-d943-4e3c-8faf-40bb0d861c7c";
/>
### TESTING INSTRUCTIONS
1. `pip install -r superset/translations/requirements.txt` (Babel 2.17.0)
and `sudo apt-get install gettext`.
2. Run `./scripts/translations/babel_update.sh` on a clean checkout of this
branch — it should produce no further changes (catalog is already in sync).
3. Confirm no target `msgid` is marked `#, fuzzy`: `grep -n -A1 '#, fuzzy'
superset/translations/<lang>/LC_MESSAGES/messages.po`
4. Validate catalogs parse: each `.po` loads via
`babel.messages.pofile.read_po` without error (no new `msgfmt --check` errors
vs. `master`).
### ADDITIONAL INFORMATION
- [ ] Has associated issue:
- [ ] Required feature flags:
- [ ] Changes UI
- [ ] Includes DB Migration (follow approval process in
[SIP-59](https://github.com/apache/superset/issues/13351))
- [ ] Migration is atomic, supports rollback & is backwards-compatible
- [ ] Confirm DB migration upgrade and downgrade tested
- [ ] Runtime estimates and downtime expectations provided
- [ ] Introduces new feature or API
- [ ] Removes existing feature or API
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
