Skip to content

feat: build Windows installer with translations embedded - #8103

Merged
nilsding merged 2 commits into
masterfrom
feature/translate-windows-installer
Apr 4, 2025
Merged

feat: build Windows installer with translations embedded#8103
nilsding merged 2 commits into
masterfrom
feature/translate-windows-installer

Conversation

@nilsding

@nilsding nilsding commented Apr 2, 2025

Copy link
Copy Markdown
Member

This PR changes the MSI to contain multiple languages, changing the installer's language to the system language.

Each additional language will add about ~1 minute of build time, as the MSI needs to be re-built for each language to create a patch/transform file. This transform file will only contain the difference from the base English installer and will be applied automatically by Windows Installer itself.

Note: we have some hardcoded strings in the Nextcloud.wxs file which won't be translated just yet. As the first step I want to make sure our builds work fine with the core translations provided by WiX, after that I'll take a look at how these can be translated (ideally with Transifex integration)

This PR requires some change in nextcloud/client-building, PR in: nextcloud/client-building#141

nilsding added 2 commits April 4, 2025 11:33
the base translations come from WixUI itself

There are still some untranslated strings in Nextcloud.wxs, we need to
take care of them at a later point.  As a first step this is good
enough.

This commit serves as a proof-of-concept, the languages I used here will
change depending on what we want to support.

Signed-off-by: Jyrki Gadinger <nilsding@nilsding.org>
building all the translated installers takes about ~1 minute each, so
keep that in mind when adding extra translations ...

Signed-off-by: Jyrki Gadinger <nilsding@nilsding.org>
@nilsding
nilsding force-pushed the feature/translate-windows-installer branch from 1632011 to a6a17e3 Compare April 4, 2025 09:33
@nilsding
nilsding enabled auto-merge April 4, 2025 09:33
@Rello Rello moved this from 🏗️ In progress to ☑️ Done in 💻 Desktop Clients team Apr 4, 2025
@github-actions

github-actions Bot commented Apr 4, 2025

Copy link
Copy Markdown
Contributor

Artifact containing the AppImage: nextcloud-appimage-pr-8103.zip

SHA256 checksum: a0b635234deddd015a749fed97dfb5819ef12039400dc3dec2648175fedae7d0

To test this change/fix you can download the above artifact file, unzip it, and run it.

Please make sure to quit your existing Nextcloud app and backup your data.

@sonarqubecloud

sonarqubecloud Bot commented Apr 4, 2025

Copy link
Copy Markdown

@nilsding
nilsding merged commit 66e9983 into master Apr 4, 2025
@nilsding
nilsding deleted the feature/translate-windows-installer branch April 4, 2025 10:17
@Rello

Rello commented Apr 7, 2025

Copy link
Copy Markdown
Collaborator

The installer will be offered in the language
EN, DE, FR, IT, ES

@github-actions

Copy link
Copy Markdown
Contributor

Hello there,
Thank you so much for taking the time and effort to create a pull request to our Nextcloud project.

We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process.

Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6

Thank you for contributing to Nextcloud and we hope to hear from you soon!

(If you believe you should not receive this message, you can add yourself to the blocklist.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Windows Installer: multiple languages, correct installer name

2 participants