[pve-devel] applied-series: [PATCH widget-toolkit/manager] fixups for ACME UI

Thomas Lamprecht t.lamprecht at proxmox.com
Thu May 7 13:54:11 CEST 2020


On 5/7/20 10:27 AM, Dominik Csapak wrote:
> this series fixes some problems and improves the ACME UI, namely:
> * auto update dc->Acme grids
> * add emptyText
> * adds missing stopUpdates
> * improve order button enable logic
> * reuse existing store for account verification
> * add auto-create/destroy feature to the diffstore
> 
> path 3/7 from manager needs the widget-toolkit patches applied,
> but all others do not need them
> 
> proxmox-widget-toolkit:
> 
> Dominik Csapak (2):
>   data/DiffStore: add autoDestroyRstore flag
>   data/DiffStore: auto-create the rstore if its just a config
> 
>  data/DiffStore.js | 35 +++++++++++++++++++++++++++++------
>  1 file changed, 29 insertions(+), 6 deletions(-)
> 
> pve-manager:
> 
> Dominik Csapak (7):
>   NodeConfig/get_acme_conf: make domains always a hash
>   ui: ACME: add emptyText and add minHeight
>   ui: dc/ACMEClusterView: use a diff/update store combo for the grids
>   ui: node/ACME: fix some eslint errors/warnings
>   ui: node/ACME: use accountselector for verification
>   ui: node/ACME: only enable order button when it should work
>   ui: node/{ACME,Certificates}: add stopUpdate on destruction
> 
>  PVE/NodeConfig.pm                  |  1 +
>  www/manager6/dc/ACMEClusterView.js | 30 +++++++++++++++---
>  www/manager6/node/ACME.js          | 50 ++++++++++++++++--------------
>  www/manager6/node/Certificates.js  |  1 +
>  4 files changed, 54 insertions(+), 28 deletions(-)
> 

applied series, the followups we talked about would be nice to have, thanks!

It's starting to get into quite a nice UX shape..




More information about the pve-devel mailing list