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