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..

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

Reply via email to