From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from firstgate.proxmox.com (firstgate.proxmox.com [212.224.123.68]) by lore.proxmox.com (Postfix) with ESMTPS id 95DE11FF179 for ; Wed, 26 Nov 2025 22:17:28 +0100 (CET) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id A58DB158F4; Wed, 26 Nov 2025 22:17:41 +0100 (CET) Message-ID: Date: Wed, 26 Nov 2025 22:17:35 +0100 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Beta To: Proxmox Datacenter Manager development discussion , Dominik Csapak References: <20251126151833.3637080-1-d.csapak@proxmox.com> Content-Language: en-US From: Thomas Lamprecht In-Reply-To: <20251126151833.3637080-1-d.csapak@proxmox.com> X-Bm-Milter-Handled: 55990f41-d878-4baa-be0a-ee34c49e34d2 X-Bm-Transport-Timestamp: 1764191818180 X-SPAM-LEVEL: Spam detection results: 0 AWL -0.022 Adjusted score from AWL reputation of From: address BAYES_00 -1.9 Bayes spam probability is 0 to 1% DMARC_MISSING 0.1 Missing DMARC policy KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_PASS -0.001 SPF: sender matches SPF record URIBL_BLOCKED 0.001 ADMINISTRATOR NOTICE: The query to URIBL was blocked. See http://wiki.apache.org/spamassassin/DnsBlocklists#dnsbl-block for more information. [fontawesome.com] Subject: Re: [pdm-devel] [PATCH datacenter-manager v5 00/26] enable custom views on the UI X-BeenThere: pdm-devel@lists.proxmox.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Proxmox Datacenter Manager development discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: Proxmox Datacenter Manager development discussion Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: pdm-devel-bounces@lists.proxmox.com Sender: "pdm-devel" Am 26.11.25 um 16:19 schrieb Dominik Csapak: > With this series, users are able to add/edit/delete custom views. > > NOTE: this series is based on lukas follow up series to the views[0] > but needs the change i mentioned in [1] to completely work. > > A new 'Views' menu entry is added where the CRUD ui sits, and an entry > for each view below it. > > The includes/excludes are editable from the CRUD grid, and the layout is > editable in the view itself. > > For now i opted to make the layout part of the `ViewConfig` struct, but > as a simple json-string only, because we're not able to use the > api-macro (for now; the rust enum features i use are currently not > supported there). To verify it regardless, we try to deserialize > the layout in the add/update call, so that no wrong layout can enter > the config (via the api). > > The patch 'ui: dashboard: prepare view for editint custom views' > is adapted from my last series, but all other changes are new, > especially the CRUD api. > > There is still one part wrong, namely > * subscriptions panels are still added by default when none is in the > layout itself (the plan is to move the subscription notice) > Smaller things I noticed: - when adding a view there is missing frontend validation for the name, one can e.g. enter "Virtual Guests" there and then only gets an error on submit. FWIW, we could split name and ID here (can be done any time later too though) - as you already write, having the subscription state of all remotes summarized in the top panel would be nice, and that should be probably computed in the backend so that it can work for views that do not have any node included. - I get where you come from with the eye icon, but maybe use "fa-plus-square-o" for now to simbolize a view with it's grid-like (well flex, but for users without CSS knowledge that distinction probably doesn't matter) arrangement of cards? https://fontawesome.com/v4/icon/plus-square-o The top-level entry should also use the same icon (in the midterm ideally same spirit but plural). - export/import would be nice and potentially cheap to add FWICT. - /view/... ACL paths are not yet visible in the ACL object path selector, e.g. when adding a new permission. - some generic resource stats counter might be nice and cheap. Might need to re-check this out with a bit fresher mind, but I already did once before and given that I saw nothing completely off for my taste after playing around with it for half an hour I see no reason to delay this any further now, especially given our current time horizon. Actually: Real nice work overall with this joint effort from you and Lukas! _______________________________________________ pdm-devel mailing list pdm-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pdm-devel