[pve-devel] applied ui: [PATCHSET v3] API Tokens

Thomas Lamprecht t.lamprecht at proxmox.com
Sun Apr 26 12:40:38 CEST 2020


On 1/21/20 1:53 PM, Fabian Grünbichler wrote:

> pve-manager:
> 
> Fabian Grünbichler (9):

>   www: add 'users' columns to Groups model
>   www: add permissions button to userview
>   www: add Token Panel + Edit Window
>   www: add Token to ACL
>   www: add TokenView with fixed userid
> 

Applied those one now too, but with some real heavy followup.
Went for a more schematic approach, albeit it could be still improved with
moving most logic into ViewControllers, but as there's currently lot's of
other stuff open for review I stopped after "getting it somewhat in shape".

As I did this for about an hour every week the last four weeks I lost a bit
the overview about specific changes.

Some are:
* lots of coding style and bug changes regarding JS / ExtJS, to you excuse:
  the user panel you started this off isn't in good shape, so a lot can be
  explained by that.
  Also, now that we have 'eslint' packaged a lot of stuff can be checked and
  fixed automatically.
* drop the "API Tokens" from the user menu, as it breaks stuff due to stacked
  windows (we normally don't do that) and z-index, also it felt a bit niche.
* partly due to aforementioned dropping I now pre-select the logged in username
  when creating a new token
* various other small UX and rendering stuff here and there

There may still come a few followups later where we get into a bit more calm
period, but for now it should do. Please test a bit around, not that I've 
missed something.

Anyway, I much more happy with a working GUI prototype for new stuff (especially
new API endpoints) than none at all - it shows that the API design was at least
used in practice, not just theoretically - the latter one are almost always a PITA
to use as fronted.

And it wasn't to bad, some where also opinionated changes of mine ;)

Thanks for this great feature!





More information about the pve-devel mailing list