You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@superset.apache.org by "FrancescoCastaldi (via GitHub)" <gi...@apache.org> on 2026/05/27 12:05:28 UTC
[PR] feat(i18n): Improve Italian (it) translation in messages.po [superset]
FrancescoCastaldi opened a new pull request, #40463:
URL: https://github.com/apache/superset/pull/40463
## Summary
This PR improves the Italian (`it`) translation file (`superset/translations/it/LC_MESSAGES/messages.po`) with more accurate and complete translations.
## Changes
- Updated and improved Italian translations in `messages.po`
- Fixed inaccurate or missing `msgstr` entries
- Improved consistency of terminology across translated strings
## Testing
- Verified that the `.po` file is syntactically valid
- Translations were reviewed for accuracy and natural Italian phrasing
## Checklist
- [x] I have read the [contributing guidelines](https://github.com/apache/superset/blob/master/CONTRIBUTING.md)
- [x] The PR title follows the [conventional commits](https://www.conventionalcommits.org/en/v1.0.0/) spec
- [x] My changes do not require new documentation
--
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
Re: [PR] fix(docs): use absolute URL for docker-builds link in 4.1 release notes [superset]
Posted by "rusackas (via GitHub)" <gi...@apache.org>.
rusackas commented on PR #40463:
URL: https://github.com/apache/superset/pull/40463#issuecomment-5227586221
Retitled this to match what's actually in the diff, this branch never had the Italian translation work the original title described, just this one-line link fix.
Held off on merging it as-is though: the link it's touching has already moved on `master` since this was opened (currently `/docs/docs/installation/docker-builds.mdx#build-presets`, not the `/docs/installation/docker-builds#build-presets` this PR's diff is based on), so applying it now would just swap in a stale target rather than fix anything. Needs a rebase against current `master` before it's mergeable. Holler if you want a hand with that, or happy to close if it's not worth chasing for a one-liner.
--
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
Re: [PR] fix(docs): use absolute URL for docker-builds link in 4.1 release notes [superset]
Posted by "FrancescoCastaldi (via GitHub)" <gi...@apache.org>.
FrancescoCastaldi closed pull request #40463: fix(docs): use absolute URL for docker-builds link in 4.1 release notes
URL: https://github.com/apache/superset/pull/40463
--
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
Re: [PR] feat(i18n): Improve Italian (it) translation in messages.po [superset]
Posted by "bito-code-review[bot] (via GitHub)" <gi...@apache.org>.
bito-code-review[bot] commented on PR #40463:
URL: https://github.com/apache/superset/pull/40463#issuecomment-4605069794
<!-- Bito Reply -->
You are correct. Hardcoding absolute URLs in documentation can lead to broken links if the domain or path structure changes. It is generally better to use relative paths or environment-specific configuration variables to ensure the documentation remains maintainable and portable.
**RELEASING/release-notes-4-1/README.md**
```
see [Docker Build Presets](/docs/installation/docker-builds/#build-presets) for more details.
```
--
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
Re: [PR] feat(i18n): Improve Italian (it) translation in messages.po [superset]
Posted by "bito-code-review[bot] (via GitHub)" <gi...@apache.org>.
bito-code-review[bot] commented on PR #40463:
URL: https://github.com/apache/superset/pull/40463#issuecomment-4554325637
AI Code Review is in progress (usually takes 3 to 15 minutes unless it's a very large PR).
--
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
Re: [PR] fix(docs): use absolute URL for docker-builds link in 4.1 release notes [superset]
Posted by "FrancescoCastaldi (via GitHub)" <gi...@apache.org>.
FrancescoCastaldi commented on PR #40463:
URL: https://github.com/apache/superset/pull/40463#issuecomment-5522390356
Thanks @rusackas and @sadpandajoe for checking! Since master already addressed this link in #30659 using relative paths and the diff here is superseded, closing this PR out. Thanks again!
--
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
Re: [PR] feat(i18n): Improve Italian (it) translation in messages.po [superset]
Posted by "rusackas (via GitHub)" <gi...@apache.org>.
rusackas commented on PR #40463:
URL: https://github.com/apache/superset/pull/40463#issuecomment-4949825357
@sadpandajoe you're right to flag this — the hardcoded URL isn't the way to go.
Turns out this can be resolved without a code change here: master already fixed this exact broken link back in October 2024 (`fix: Link Checking` #30659), using a relative link (`/docs/docs/installation/docker-builds.mdx#build-presets`) rather than a hardcoded one. This branch is quite far behind master at this point (~5,300 commits), and once rebased, this specific line's diff would disappear entirely since master's current version is already the correct fix.
Given the PR's remaining diff (relative to current master) is just this one now-superseded line, I'd suggest closing this one out — happy to reopen/help rebase if there's other content in here worth salvaging.
--
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
Re: [PR] feat(i18n): Improve Italian (it) translation in messages.po [superset]
Posted by "netlify[bot] (via GitHub)" <gi...@apache.org>.
netlify[bot] commented on PR #40463:
URL: https://github.com/apache/superset/pull/40463#issuecomment-4554354766
### <span aria-hidden="true">✅</span> Deploy Preview for *superset-docs-preview* ready!
| Name | Link |
|:-:|------------------------|
|<span aria-hidden="true">🔨</span> Latest commit | f6e460fc0e77c670f9d4830289125416f8ba74fd |
|<span aria-hidden="true">🔍</span> Latest deploy log | https://app.netlify.com/projects/superset-docs-preview/deploys/6a16de0acd820b0008f79049 |
|<span aria-hidden="true">😎</span> Deploy Preview | [https://deploy-preview-40463--superset-docs-preview.netlify.app](https://deploy-preview-40463--superset-docs-preview.netlify.app) |
|<span aria-hidden="true">📱</span> Preview on mobile | <details><summary> Toggle QR Code... </summary><br /><br /><br /><br />_Use your smartphone camera to open QR code link._</details> |
|<span aria-hidden="true">🤖</span> Make changes | [Run an agent on this branch](https://app.netlify.com/projects/superset-docs-preview/agent-runs#q1YqKMrPLShRslJS0lFKKkrMS84Ast0yKzLz0nVzMvOyi4HiKakFOfmVnilAGbNEQ7OUVIPE5BQLI4MkAwMDizRzSwMTS6VaAA) |
---
<!-- [superset-docs-preview Preview](https://deploy-preview-40463--superset-docs-preview.netlify.app) -->
_To edit notification comments on pull requests, go to your [Netlify project configuration](https://app.netlify.com/projects/superset-docs-preview/configuration/notifications#deploy-notifications)._
--
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
Re: [PR] feat(i18n): Improve Italian (it) translation in messages.po [superset]
Posted by "sadpandajoe (via GitHub)" <gi...@apache.org>.
sadpandajoe commented on code in PR #40463:
URL: https://github.com/apache/superset/pull/40463#discussion_r3343131858
##########
RELEASING/release-notes-4-1/README.md:
##########
@@ -137,4 +137,4 @@ There is now a [metadata bar](https://github.com/apache/superset/pull/27857) add
## Change to Docker image builds
-Starting in 4.1.0, the release's docker image does not ship with drivers needed to operate Superset. Users may need to install a driver for their metadata database (MySQL or Postgres) as well as the driver for their data warehouse. This is a result of changes to the `lean` docker image that official releases come from; see [Docker Build Presets](/docs/docs/installation/docker-builds.mdx#build-presets) for more details.
+Starting in 4.1.0, the release's docker image does not ship with drivers needed to operate Superset. Users may need to install a driver for their metadata database (MySQL or Postgres) as well as the driver for their data warehouse. This is a result of changes to the `lean` docker image that official releases come from; see [Docker Build Presets](https://superset.apache.org/docs/installation/docker-builds/#build-presets) for more details.
Review Comment:
We don't want to hardcode the url, right?
--
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