Testing

[rodinux]

Problem

• Some users need edit the mail config

Solution

• move the mail config in the folder config.local.user.php

PR Status

• [ ] Code finished and ready to be reviewed/tested
• [ ] The fix/enhancement were manually tested (if applicable)

Automatic tests

Automatic tests can be triggered on https://ci-apps-dev.yunohost.org/ after creating the PR, by commenting "!testme", "!gogogadgetoci" or "By the power of systemd, I invoke The Great App CI to test this Pull Request!". (N.B. : for this to work you need to be a member of the Yunohost-Apps organization)

[rodinux]

!testme

[yunohost-bot]

:carousel_horse:

[yunohost-bot]

:worm:

[rodinux]

!testme

[yunohost-bot]

:rocket:

[yunohost-bot]

:books: :bug:

[rodinux]

@alexAubin @ericgaspar , can I submit you some questions about the mail config ? Some users have to change parameters, there is perhaps something to improve. For exemple here https://forum.yunohost.org/t/paheko-erp-for-non-profit-organization/23699/75 I don't understand what the consequence of using something like

const SMTP_HOST = false; ----> utilisation de localhost comme adresse de serveur de courrier dans Paheko (cf doc Paheko)
const SMTP_PORT = 25;
const SMTP_USER = ‘paheko’; —> C’est le user créé lors de l’installation de paheko
const SMTP_PASSWORD = ###hidden###‘’;
const SMTP_SECURITY = ‘STARTTLS’;
const SMTP_HELO_HOSTNAME = ‘__DOMAIN__’;—> c’est le sous-domaine sur lequel eqt installé Paheko
const MAIL_RETURN_PATH = ‘__APP__@__DOMAIN__’;
const MAIL_SENDER = ‘__APP__@FQDN_DOMAIN’;

• first, does use the SMTP_HOST = false, make just use the mail php() instead of the the config postfix app_senders
• Is it possible to use as here the FQDN_DOMAIN ? I mean the default domain of a yunohost ?

Perhaps it would be better integrate a yunohost user in the install to be fully compatible, it means found how add it to the post-install process also, not easy to code, but I am not sure of this

[rodinux]

!testme

[yunohost-bot]

:carousel_horse:

[yunohost-bot]

:books: :bug:

[rodinux]

related to fix #56 and #55 Is it ok ? one rewiew required ? @fflorent @alexAubin @ericgaspar @