Skip to content

IONOS(mail): update submodule to v5.12.3 IONOS port (NC33, NSW-1070) - #387

Draft
printminion-co wants to merge 1 commit into
ionos-dev-v33from
mk/dev/NSW-1070-update-mail-nc33
Draft

printminion-co wants to merge 1 commit into
ionos-dev-v33from
mk/dev/NSW-1070-update-mail-nc33

Conversation

@printminion-co

@printminion-co printminion-co commented Oct 5, 2026 •

Copy link
Copy Markdown

Moves apps-external/mail from v5.5.11 + IONOS to upstream nextcloud/mail v5.12.3 + IONOS, so the whole build and the resulting artifact can be checked before the ncw-mail stack is merged.

Important

Draft, do not merge yet. The gitlink points at 9fdc208, the head of the unmerged ncw-mail stack:
IONOS-Productivity/ncw-mail#61 → #62 → #63 (base ionos-dev-5.12).
Once the stack is merged, I'll replace this pointer with the release tag v5.12.3-<sha7> and mark the PR ready.

What changes for mail

  • Upstream jump: v5.5.11 → v5.12.3. info.xml declares NC 32–35. The upgrade runs the upstream migrations from 5.5.11 on.
  • IONOS delta: all 148 IONOS commits on the live ionos-dev are replayed (NSW-782 from draft PR IONOS(collectives): add submodule v3.2.0 (22f7609) #55 is not included). Six that are obsolete or already upstream were dropped; the ncw-mail PR descriptions list every one.
  • New: the allow_new_mail_aliases admin toggle, doc/ionos-admin.md, and regenerated IONOS translations. The regeneration also fixes the plural translations that Phrase had broken on ionos-dev.
  • Build: no IONOS/Makefile change needed.
    • Node goes from ^22 to ^24 (and npm from ^10.5 to ^11.3). This repo's package.json already declares ^24, which the build job uses.
    • mail now runs a patch-package postinstall during npm ci.
    • The private ionos-mail-configuration-api-client composer package is still required, same as before.

How to check

  1. Watch the build-artifact run on this PR. In particular, the build_mail_app matrix job must pass.
  2. Spin up the artifact with .dev/container/check_release.sh pr-387 ncw.
  3. Check:
    • occ upgrade from an existing 5.5.11 instance completes.
    • occ mail:provider:list and occ mail:provider:status ionos work.
    • Admin settings → Mail provider accounts lists mailboxes.
    • With allow_new_mail_aliases=no, the "Add alias" button is hidden.
    • DE/FR strings are translated.

Locally I already ran the 5.5.11 → 5.12.3 upgrade on a throwaway ionos-dev-v33 install with SQLite. It passed, with the production-style mixed allow_new_mail_aliases=no intact afterwards and alias creation blocked.

Unrelated to the open fork-app sweep #381, which doesn't touch mail.

Jira: NSW-1070

@printminion-co
printminion-co force-pushed the mk/dev/NSW-1070-update-mail-nc33 branch 2 times, most recently from f01402e to 24e05ae Compare October 13, 2026 08:09
Moves apps-external/mail from v5.5.11 + IONOS (1627f0972) to upstream
nextcloud/mail v5.12.3 + IONOS, so the whole build can be checked
before the ncw-mail stack is merged.

Temporary pointer: the SHA is the head of the unmerged ncw-mail PR
stack #61 -> #62 -> #63. It is replaced by the release tag
v5.12.3-<sha7> once that stack is merged into ionos-dev-5.12.

SHA: 9fdc20866025bd99b4d526aead3d6e0c32218033

Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
@printminion-co
printminion-co force-pushed the mk/dev/NSW-1070-update-mail-nc33 branch from 24e05ae to 57f5876 Compare October 14, 2026 06:52

This branch has not been deployed

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant