Re: [PR] fix(table charts v1 & v2): pagination, translation and search by [superset]

["github-actions[bot] (via GitHub)" <gi...@apache.org>]

github-actions[bot] commented on PR #36881:
URL: https://github.com/apache/superset/pull/36881#issuecomment-4648037612

   <!-- translation-regression-bot -->
   ## ⚠️ Translation Regression Detected
   
   A source change in this PR renamed or reworded strings, invalidating existing translations (they are now `#, fuzzy`) in `ar`, `ca`, `cs`, `de`, `fi`, `fr`, `it`, `ja`, `ko`, `lv`, `mi`, `nl`, `pt`, `ru`, `sk`, `sl`, `th`, `tr`, `uk`. Please resolve the affected `.po` files before merging.
   
   _Note: intentionally **deleting** a translatable string is not a regression and is not flagged here — only translations invalidated by a renamed/reworded source string are._
   
   | Language | Fuzzy before | Fuzzy after | New |
   |----------|-------------:|------------:|----:|
   | `ar` | 1136 | 1137 | +1 |
   | `ca` | 815 | 816 | +1 |
   | `cs` | 787 | 788 | +1 |
   | `de` | 997 | 998 | +1 |
   | `fi` | 4823 | 4824 | +1 |
   | `fr` | 1014 | 1015 | +1 |
   | `it` | 1667 | 1668 | +1 |
   | `ja` | 322 | 323 | +1 |
   | `ko` | 1521 | 1522 | +1 |
   | `lv` | 295 | 296 | +1 |
   | `mi` | 807 | 808 | +1 |
   | `nl` | 1143 | 1144 | +1 |
   | `pt` | 1839 | 1840 | +1 |
   | `ru` | 293 | 294 | +1 |
   | `sk` | 774 | 775 | +1 |
   | `sl` | 963 | 964 | +1 |
   | `th` | 4823 | 4824 | +1 |
   | `tr` | 786 | 787 | +1 |
   | `uk` | 293 | 294 | +1 |
   
   ### How to fix
   
   **1. Install dependencies** (if not already set up):
   
   ```bash
   pip install -r superset/translations/requirements.txt
   sudo apt-get install gettext   # or: brew install gettext
   ```
   
   **2. Re-extract strings and sync `.po` files:**
   
   ```bash
   ./scripts/translations/babel_update.sh
   ```
   
   This rewrites `superset/translations/messages.pot` from the current source files and merges the changes into every `.po` file. Strings whose `msgid` changed will be marked `#, fuzzy`.
   
   **3. Resolve the fuzzy entries** in the affected language files (`ar`, `ca`, `cs`, `de`, `fi`, `fr`, `it`, `ja`, `ko`, `lv`, `mi`, `nl`, `pt`, `ru`, `sk`, `sl`, `th`, `tr`, `uk`):
   
   ```bash
   grep -n '#, fuzzy' superset/translations/<lang>/LC_MESSAGES/messages.po
   ```
   
   For each fuzzy entry, either rewrite the `msgstr` to match the new string and remove the `#, fuzzy` line, or clear the `msgstr` to `""` if you cannot provide a translation.
   
   **4. Commit your changes to the `.po` files.**
   


-- 
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: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org