antalotec opened a new pull request, #45973:
URL: https://github.com/apache/superset/pull/45973

   ### SUMMARY
   
   `t('Log')` is the resource name of the **Action Log** list — the record of 
what each user
   did. On its own, out of the surrounding code, the word is ambiguous: it is 
also a
   logarithm and, for a reader in a hurry, a login. Two catalogs picked the 
wrong sense:
   
   | | `msgstr` | meaning |
   |---|---|---|
   | `it` | `Login` | a sign-in, not a record |
   | `ca` | `logarítmic` | the mathematical logarithm |
   
   This adds an `i18n:` translator note on the call, using the mechanism added 
in #44397, and
   corrects those two entries. The corrections are not a judgement call: each 
catalog already
   translates the related `Logs` entry correctly — `it` as *Log*, `ca` as 
*Registres* — so the
   singular is simply made consistent with the plural the same translators 
chose.
   
   ### What is in the diff
   
   - `superset-frontend/src/pages/ActionLog/index.tsx` — the `i18n:` note.
   - `superset/translations/messages.pot` and the 30 catalogs — the note 
propagated by
     `babel_update.sh`, two lines each.
   - `it` and `ca` — the corrected `msgstr`.
   
   Only the lines belonging to this change are included. Running 
`babel_update.sh` on current
   `master` also rewrites ~970 unrelated lines across 31 files, because the 
catalogs have
   drifted from the template since the last sync; that drift is pre-existing 
and has been
   deliberately left out of this PR.
   
   Follow-up to #45953, which repaired malformed format specifiers in three of 
these
   catalogs.
   
   ### TESTING INSTRUCTIONS
   
   ```bash
   ./scripts/translations/babel_update.sh
   python scripts/translations/check_pot_drift.py     # template matches a 
fresh extraction
   grep -B3 '^msgid "Log"$' superset/translations/messages.pot
   ```
   
   The note must appear on the entry — a misplaced `i18n:` comment is silently 
dropped, so
   this was verified rather than assumed.
   
   Checks run locally, all green:
   
   - `npx jest src/pages/ActionLog/ActionLog.test.tsx` — 6/6
   - `npm run lint` (oxlint) — clean · `oxfmt --check` — clean
   - `python scripts/translations/check_pot_drift.py` — matches
   - `msgfmt --check` on `it`, `pt`, `fr` — unchanged from master (1 each, from 
an unrelated
     source-string defect), so this introduces no new errors
   
   ### ADDITIONAL INFORMATION
   
   - [ ] Has associated issue:
   - [ ] Required feature flags:
   - [x] Changes UI
   - [ ] Includes DB Migration
   - [ ] Introduces new feature or API
   - [ ] Removes existing feature or API
   
   ---
   
   Developed at MYSPHERA, S.L.
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


-- 
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]

  • [PR] fix(i1... antalotec (via GitHub) via notifications
    • Re: [P... codeant-ai-for-open-source[bot] (via GitHub) via notifications
    • Re: [P... codecov[bot] (via GitHub) via notifications
    • Re: [P... antalotec (via GitHub) via notifications

Reply via email to