all lists on lists.proxmox.com
 help / color / mirror / Atom feed
From: "Shannon Sterz" <s.sterz@proxmox.com>
To: "Shannon Sterz" <s.sterz@proxmox.com>
Cc: pdm-devel@lists.proxmox.com
Subject: Re: [pdm-devel] [PATCH datacenter-manager v3 1/3] ui: add a token panel and a token acl edit menu in the permissions panel
Date: Fri, 17 Oct 2025 14:52:50 +0200	[thread overview]
Message-ID: <DDKM18KQL2DV.1VB0HCZN6LC97@proxmox.com> (raw)
In-Reply-To: <20251017124646.294343-7-s.sterz@proxmox.com>

On Fri Oct 17, 2025 at 2:46 PM CEST, Shannon Sterz wrote:
> this allows users to create and manage api tokens.
>
> Signed-off-by: Shannon Sterz <s.sterz@proxmox.com>

this should have:

Reviewed-by: Lukas Wagner <l.wagner@proxmox.com>
Tested-by: Lukas Wagner <l.wagner@proxmox.com>

sorry i forgot to add that initially.

> ---
>  ui/src/configuration/mod.rs | 33 +++++++++++++++++++++++++++------
>  1 file changed, 27 insertions(+), 6 deletions(-)
>
> diff --git a/ui/src/configuration/mod.rs b/ui/src/configuration/mod.rs
> index 0a92796..4110f39 100644
> --- a/ui/src/configuration/mod.rs
> +++ b/ui/src/configuration/mod.rs
> @@ -7,7 +7,7 @@ use pwt::widget::{Container, MiniScrollMode, Panel, TabBarItem, TabPanel};
>  use proxmox_yew_comp::configuration::TimePanel;
>  use proxmox_yew_comp::configuration::{DnsPanel, NetworkView};
>  use proxmox_yew_comp::tfa::TfaView;
> -use proxmox_yew_comp::{AclEdit, AclView, AuthView, UserPanel};
> +use proxmox_yew_comp::{AclEdit, AclView, AuthView, TokenPanel, UserPanel};
>
>  mod permission_path_selector;
>  mod webauthn;
> @@ -73,6 +73,19 @@ pub fn access_control() -> Html {
>                      .into()
>              },
>          )
> +        .with_item_builder(
> +            TabBarItem::new()
> +                .key("api-tokens")
> +                .icon_class("fa fa-user-o")
> +                .label(tr!("API Token")),
> +            |_| {
> +                Container::new()
> +                    .class("pwt-content-spacer")
> +                    .class(pwt::css::FlexFit)
> +                    .with_child(TokenPanel::new())
> +                    .into()
> +            },
> +        )
>          .with_item_builder(
>              TabBarItem::new()
>                  .key("two-factor")
> @@ -95,11 +108,19 @@ pub fn access_control() -> Html {
>                  Container::new()
>                      .class("pwt-content-spacer")
>                      .class(pwt::css::FlexFit)
> -                    .with_child(AclView::new().with_acl_edit_menu_entry(
> -                        tr!("User Permission"),
> -                        "fa fa-fw fa-user",
> -                        acl_edit.clone().use_tokens(false),
> -                    ))
> +                    .with_child(
> +                        AclView::new()
> +                            .with_acl_edit_menu_entry(
> +                                tr!("User Permission"),
> +                                "fa fa-fw fa-user",
> +                                acl_edit.clone().use_tokens(false),
> +                            )
> +                            .with_acl_edit_menu_entry(
> +                                tr!("Token Permission"),
> +                                "fa fa-fw fa-user-o",
> +                                acl_edit.clone().use_tokens(true),
> +                            ),
> +                    )
>                      .into()
>              },
>          )



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


  reply	other threads:[~2025-10-17 12:53 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-10-17 12:46 [pdm-devel] [PATCH datacenter-manager/yew-comp/yew-widget-toolkit v3 0/7] add better token support for pdm Shannon Sterz
2025-10-17 12:46 ` [pdm-devel] [PATCH yew-widget-toolkit v3 1/1] props: add readonly to field_std_props Shannon Sterz
2025-10-21 18:52   ` [pdm-devel] applied: " Thomas Lamprecht
2025-10-17 12:46 ` [pdm-devel] [PATCH yew-comp v3 1/3] utils/tfa add recover/token panel: add copy_text_to_clipboard function Shannon Sterz
2025-10-22 17:03   ` [pdm-devel] applied: " Thomas Lamprecht
2025-10-17 12:46 ` [pdm-devel] [PATCH yew-comp v3 2/3] token panel: improve token secret dialog layout and hide password Shannon Sterz
2025-10-22 17:03   ` [pdm-devel] applied: " Thomas Lamprecht
2025-10-17 12:46 ` [pdm-devel] [PATCH yew-comp v3 3/3] utils: split out clipboard helpers into their own modules Shannon Sterz
2025-10-17 12:51   ` Shannon Sterz
2025-10-22 17:03   ` [pdm-devel] applied: " Thomas Lamprecht
2025-10-17 12:46 ` [pdm-devel] [PATCH datacenter-manager v3 1/3] ui: add a token panel and a token acl edit menu in the permissions panel Shannon Sterz
2025-10-17 12:52   ` Shannon Sterz [this message]
2025-10-22 17:22   ` [pdm-devel] applied: " Thomas Lamprecht
2025-10-17 12:46 ` [pdm-devel] [PATCH datacenter-manager v3 2/3] server: access: use token endpoints from proxmox-access-control Shannon Sterz
2025-10-17 12:52   ` Shannon Sterz
2025-10-22 17:22   ` [pdm-devel] applied: " Thomas Lamprecht
2025-10-17 12:46 ` [pdm-devel] [PATCH datacenter-manager v3 3/3] server: clean up acl tree entries and api tokens when deleting users Shannon Sterz
2025-10-17 12:52   ` Shannon Sterz
2025-10-22 17:22   ` [pdm-devel] applied: " Thomas Lamprecht

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=DDKM18KQL2DV.1VB0HCZN6LC97@proxmox.com \
    --to=s.sterz@proxmox.com \
    --cc=pdm-devel@lists.proxmox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal