[pve-devel] applied: [PATCH manager v3] ui: dc/Auth: add sync button

Thomas Lamprecht t.lamprecht at proxmox.com
Wed Apr 29 16:49:46 CEST 2020


On 4/29/20 2:32 PM, Dominik Csapak wrote:
> opens a window with the parameters for the sync and two buttons:
> 'preview' and 'sync'
> 
> both open the taskviewer, but the 'preview' one sets the 'dry-run'
> parameter so that it does not get written out to the user.cfg
> 
> loads the realm config and prefills the selection with values from
> the config, and shows a hint about where to set the defaults
> if none are set
> 
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> ---
> changes from v2:
> * drop the '(from Config)' and '(Default)' options
> * show a hint about where to set default options
> * show window after closing the task of a preview
> 
>  www/manager6/Makefile         |   1 +
>  www/manager6/dc/AuthView.js   |  21 ++++
>  www/manager6/dc/SyncWindow.js | 186 ++++++++++++++++++++++++++++++++++
>  3 files changed, 208 insertions(+)
>  create mode 100644 www/manager6/dc/SyncWindow.js
> 

Applied, thanks! I added some emptyText "No default available", we could maybe
disable "Purge ACLs" if "Full" is not set to yes, but not sure if that's clear
or worth it.. Anyway, thanks!





More information about the pve-devel mailing list