On 24.06.21 10:17, Dietmar Maurer wrote:
> ---
>  PVE/HTTPServer.pm                  |   4 +-
>  www/manager6/Utils.js              |   8 +++
>  www/manager6/window/LoginWindow.js | 105 ++++++++++++++++++++++++++++-
>  3 files changed, 114 insertions(+), 3 deletions(-)
> 
>

applied, thanks!

just FYI:
I squashed in a few fixes though, for one the gettext usage was broken as
things like `gettext('' + variable)` cannot work (would never wind a 
translation)
and the scanner in proxmox-i18n cannot detect such things either.

One must use either `Ext.String.format((gettext("bla bla: {0}"), variable)` or 
add
the variable after the gettext call, depending on use-case.

Also factored out the `openid_login_param` helper to proxmox-widget toolkit
as `getOpenIDRedirectionAuthorization`, also fixed some casing (we prefer 
camelCase
over snake_case in JS for new code[0]).

[0]: https://pve.proxmox.com/wiki/Javascript_Style_Guide#Casing


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

Reply via email to