Skip to content

feat(config): disable mail configuration for brand 'ionoscloud'#132

Merged
Arsalanulhaq merged 1 commit intomainfrom
as/feat/disable-mailconfig-when-brand-is-ionoscloud
Mar 11, 2026
Merged

feat(config): disable mail configuration for brand 'ionoscloud'#132
Arsalanulhaq merged 1 commit intomainfrom
as/feat/disable-mailconfig-when-brand-is-ionoscloud

Conversation

@Arsalanulhaq
Copy link
Contributor

This pull request adds logic to the configure_ionos_mailconfig_api function in configure.sh to conditionally disable the mail configuration based on the system brand. If the brand is set to "ionoscloud", the script will automatically disable the ionos-mailconfig-enabled setting.

Brand-specific configuration:

  • Added a check for the brand system configuration; if it is "ionoscloud", the script disables the ionos-mailconfig-enabled setting to prevent mail configuration from being enabled for this brand.

@Arsalanulhaq Arsalanulhaq force-pushed the as/feat/disable-mailconfig-when-brand-is-ionoscloud branch from ae531f6 to cd9c215 Compare March 10, 2026 14:18
@Arsalanulhaq Arsalanulhaq requested a review from Copilot March 10, 2026 14:51
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR aims to prevent enabling the Nextcloud Mail “IONOS mailconfig” integration for the ionoscloud brand by adding brand-conditional logic to configure_ionos_mailconfig_api in configure.sh.

Changes:

  • Adds a brand check intended to disable mail ionos-mailconfig-enabled when the system is running under the ionoscloud brand.
  • Forces ionos-mailconfig-enabled back to no when the brand condition matches.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Arsalanulhaq Arsalanulhaq force-pushed the as/feat/disable-mailconfig-when-brand-is-ionoscloud branch from cd9c215 to f32570d Compare March 10, 2026 16:11
@Arsalanulhaq Arsalanulhaq force-pushed the as/feat/disable-mailconfig-when-brand-is-ionoscloud branch from f32570d to 000d353 Compare March 11, 2026 09:10
@Arsalanulhaq Arsalanulhaq added this to the ncw-5 milestone Mar 11, 2026
Signed-off-by: Arsalan Ul Haq Sohni <arsalan-ul-haq.sohni@strato.de>
@printminion-co printminion-co force-pushed the as/feat/disable-mailconfig-when-brand-is-ionoscloud branch from 000d353 to 9fc57d4 Compare March 11, 2026 10:09
@Arsalanulhaq Arsalanulhaq merged commit 02e3070 into main Mar 11, 2026
12 checks passed
@Arsalanulhaq Arsalanulhaq deleted the as/feat/disable-mailconfig-when-brand-is-ionoscloud branch March 11, 2026 10:11
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.

3 participants