Am 04.12.24 um 12:37 schrieb Fiona Ebner:
> Reported in the community forum:
> https://forum.proxmox.com/threads/158518/
> 
> Signed-off-by: Fiona Ebner <f.eb...@proxmox.com>
> ---
>  www/manager6/dc/UserView.js | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/www/manager6/dc/UserView.js b/www/manager6/dc/UserView.js
> index 82bd2ee7..d5844674 100644
> --- a/www/manager6/dc/UserView.js
> +++ b/www/manager6/dc/UserView.js
> @@ -68,10 +68,16 @@ Ext.define('PVE.dc.UserView', {
>               return false;
>           },
>           handler: function(btn, event, rec) {
> +             let hintHtml;
> +             if (rec.data['realm-type'] === 'pam') {
> +                 hintHtml = gettext("For the PAM realm, this only applies to 
> the local node.");

I'd switch the place of `only` and `applies` and maybe s/local/connected/,
like:

For the PAM realm, this applies only to the connected node.

> +             }
> +
>               Ext.create('Proxmox.window.PasswordEdit', {
>                   userid: rec.data.userid,
>                   confirmCurrentPassword: Proxmox.UserName !== 'root@pam',
>                   autoShow: true,
> +                 hintHtml: hintHtml,

FYI, if variable name and property name match, i.e., left part of the colon is
the same as the right part, you can just use that directly, e.g. `{ hintHtml }`
is the same as `{ hintHtml: hintHtml }`; but not something that needs to be
used as of now, but would be fine to be used.

>                   minLength: 8,
>                   listeners: {
>                       destroy: () => reload(),



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to