[pmg-devel] [PATCH pmg-gui 2/2] mail-proxy-relaying: add help to Default Relay

Stoiko Ivanov s.ivanov at proxmox.com
Tue Sep 12 16:22:54 CEST 2023


Thanks for the addition - LGTM in general.

One question remains - why only add it here and not also the the other
editors (port, protocol, nomx)?


On Thu,  7 Sep 2023 15:47:04 +0200
Maximiliano Sandoval <m.sandoval at proxmox.com> wrote:

> Add link to [1]. This is quite an important setting, might as well
> provide documentation.
> 
> [1] https://pmg.proxmox.com/pmg-docs/chapter-pmgconfig.html#pmgconfig_mailproxy_relaying
> 
> Signed-off-by: Maximiliano Sandoval <m.sandoval at proxmox.com>
> ---
>  js/MailProxyRelaying.js | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/js/MailProxyRelaying.js b/js/MailProxyRelaying.js
> index 411c9b4..c9fafbe 100644
> --- a/js/MailProxyRelaying.js
> +++ b/js/MailProxyRelaying.js
> @@ -10,6 +10,7 @@ Ext.define('PMG.MailProxyRelaying', {
>  	me.add_text_row('relay', gettext('Default Relay'), {
>  	    deleteEmpty: true,
>  	    defaultValue: Proxmox.Utils.noneText,
> +	    onlineHelp: 'pmgconfig_mailproxy_relaying',
>  	});
>  
>  	me.add_integer_row('relayport', gettext('Relay Port'), {





More information about the pmg-devel mailing list