Feedback email recipient(s) : add flexibility

[Nicolas Micoud]

Hello,

ATM, when a user uses the Feedback email dialog, the recipient is read from the System.SupportEMail.

In my SaaS environnement, it is not convenient as we are using 3 languages, so I would like to be able to set an adress (To) per tenant and a Cc (or Bcc) to another one.

So, my idea is to :

Add 3 SysConfig keys (Org level) :

FEEDBACK_EMAIL_TO
FEEDBACK_EMAIL_CC
FEEDBACK_EMAIL_BCC

Replace DefaultFeedbackService.EmailSupportAction.showEmailDialog

if (!Util.isEmpty(system.getSupportEMail()))
{
    dialog.addTo(system.getSupportEMail(), true);
}

with values of the SysConfig keys.

The migration script should initialize FEEDBACK_EMAIL_TO using the system.getSupportEMail() value.

wdyt ?

Regards,

Nicolas

[Heng Sin Low]

I think it might confuse existing users if we are having both in the system and changes to system.getSupportEMail()  being silently ignored (after migration). Perhaps we can have the 3 new sys config initially empty and take priority over system.getSupportEMail when it is filled.

--
You received this message because you are subscribed to the Google Groups "iDempiere" group.
To unsubscribe from this group and stop receiving emails from it, send an email to idempiere+...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/idempiere/6f5a9dd8-273a-4583-a19e-5f67ae74f76do%40googlegroups.com.

[Nicolas Micoud]

https://idempiere.atlassian.net/browse/IDEMPIERE-4461