all lists on lists.proxmox.com
 help / color / mirror / Atom feed
* [pdm-devel] [PATCH proxmox-datacenter-manager 1/1] resource tree: fix navigating to network resources
@ 2025-12-01 17:57 Stefan Hanreich
  2025-12-01 18:27 ` [pdm-devel] applied: " Thomas Lamprecht
  0 siblings, 1 reply; 2+ messages in thread
From: Stefan Hanreich @ 2025-12-01 17:57 UTC (permalink / raw)
  To: pdm-devel

The resources tree generated `sdn/zones` as a slug for the zone
network resource, which shows the SDN resource tree - but the actual
corect slug would be `sdn`. Additionally, with the addition of
fabrics to the network resources, make the resource tree navigate to
the SDN tree as well, instead of just for zones.

Signed-off-by: Stefan Hanreich <s.hanreich@proxmox.com>
---
 ui/src/lib.rs | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/ui/src/lib.rs b/ui/src/lib.rs
index 27a5108..02cebed 100644
--- a/ui/src/lib.rs
+++ b/ui/src/lib.rs
@@ -1,6 +1,6 @@
 use js_sys::{Array, JsString, Object};
 use pdm_api_types::remotes::RemoteType;
-use pdm_api_types::resource::{PveLxcResource, PveNetworkResource, PveQemuResource};
+use pdm_api_types::resource::{PveLxcResource, PveQemuResource};
 use pdm_client::types::Resource;
 use serde::{Deserialize, Serialize};
 
@@ -185,9 +185,7 @@ pub(crate) fn navigate_to<C: yew::Component>(
                         true,
                         format!("storage+{}+{}", storage.node, storage.storage),
                     ),
-                    pdm_client::types::Resource::PveNetwork(PveNetworkResource::Zone(_)) => {
-                        (false, "sdn/zones".to_string())
-                    }
+                    pdm_client::types::Resource::PveNetwork(_) => (false, "sdn".to_string()),
                     pdm_client::types::Resource::PbsDatastore(store) => (true, store.name.clone()),
                     // FIXME: implement
                     _ => return None,
-- 
2.47.3


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


^ permalink raw reply	[flat|nested] 2+ messages in thread

* [pdm-devel] applied: [PATCH proxmox-datacenter-manager 1/1] resource tree: fix navigating to network resources
  2025-12-01 17:57 [pdm-devel] [PATCH proxmox-datacenter-manager 1/1] resource tree: fix navigating to network resources Stefan Hanreich
@ 2025-12-01 18:27 ` Thomas Lamprecht
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas Lamprecht @ 2025-12-01 18:27 UTC (permalink / raw)
  To: pdm-devel, Stefan Hanreich

On Mon, 01 Dec 2025 18:57:22 +0100, Stefan Hanreich wrote:
> The resources tree generated `sdn/zones` as a slug for the zone
> network resource, which shows the SDN resource tree - but the actual
> corect slug would be `sdn`. Additionally, with the addition of
> fabrics to the network resources, make the resource tree navigate to
> the SDN tree as well, instead of just for zones.
> 
> 
> [...]

Applied, thanks!

[1/1] resource tree: fix navigating to network resources
      commit: 11f26b0dd0305154629655108402fd67e32303d8


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


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2025-12-01 18:26 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-12-01 17:57 [pdm-devel] [PATCH proxmox-datacenter-manager 1/1] resource tree: fix navigating to network resources Stefan Hanreich
2025-12-01 18:27 ` [pdm-devel] applied: " Thomas Lamprecht

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