all lists on lists.proxmox.com
 help / color / mirror / Atom feed
* [PATCH datacenter-manager] ui: tree-wide: fix spelling of show-task_progress() function
@ 2026-06-17 12:41 Shannon Sterz
  2026-06-17 13:46 ` Dominik Csapak
  2026-06-18 20:08 ` applied: " Thomas Lamprecht
  0 siblings, 2 replies; 3+ messages in thread
From: Shannon Sterz @ 2026-06-17 12:41 UTC (permalink / raw)
  To: pdm-devel

the commit 51a9489dd418 (loadable component: fix show_task_progress
method name spelling) fixed the name of this function. adapt to it so
pdm ui will build against yew-comp once this change is bumped.

Signed-off-by: Shannon Sterz <s.sterz@proxmox.com>
---

Notes:
    i know the change this is adapting to isn't bumped yet, so applying it
    before that will break the build. however, i could not find anything
    like this on list [1] so i thought i'll send a fix for this before
    it's accidentally missed.
    
    [1]: https://lore.proxmox.com/all/?q=show_task_progres

 ui/src/configuration/subscription_registry.rs | 2 +-
 ui/src/guests.rs                              | 2 +-
 ui/src/pve/tree.rs                            | 6 +++---
 ui/src/remotes/updates.rs                     | 2 +-
 ui/src/widget/snapshot_window.rs              | 2 +-
 5 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/ui/src/configuration/subscription_registry.rs b/ui/src/configuration/subscription_registry.rs
index 470789bc..32343c15 100644
--- a/ui/src/configuration/subscription_registry.rs
+++ b/ui/src/configuration/subscription_registry.rs
@@ -885,7 +885,7 @@ impl LoadableComponent for SubscriptionRegistryComp {
                         Ok(None) => link.change_view(None),
                         Ok(Some(upid)) => {
                             link.change_view(None);
-                            link.show_task_progres(upid);
+                            link.show_task_progress(upid);
                         }
                         Err(err) => link.show_error(tr!("Apply Pending"), err.to_string(), true),
                     }
diff --git a/ui/src/guests.rs b/ui/src/guests.rs
index f11f0de5..ef670a00 100644
--- a/ui/src/guests.rs
+++ b/ui/src/guests.rs
@@ -340,7 +340,7 @@ impl LoadableComponent for GuestPanelComp {
                 // another remote that ran concurrently cannot point the task
                 // viewer at the wrong remote
                 self.set_task_base_url(format!("/pve/remotes/{}/tasks", upid.remote()).into());
-                ctx.link().show_task_progres(upid.to_string());
+                ctx.link().show_task_progress(upid.to_string());
             }
             Msg::GuestAction(action, key) => {
                 let Some(entry) = self.store.read().lookup_record(&key).cloned() else {
diff --git a/ui/src/pve/tree.rs b/ui/src/pve/tree.rs
index 7a14a05b..0042331c 100644
--- a/ui/src/pve/tree.rs
+++ b/ui/src/pve/tree.rs
@@ -362,7 +362,7 @@ impl LoadableComponent for PveTreeComp {
                         };
 
                         match res {
-                            Ok(upid) => link.show_task_progres(upid.to_string()),
+                            Ok(upid) => link.show_task_progress(upid.to_string()),
                             Err(err) => link.show_error(tr!("Error"), err.to_string(), true),
                         }
                     }),
@@ -386,7 +386,7 @@ impl LoadableComponent for PveTreeComp {
                         };
 
                         match res {
-                            Ok(upid) => link.show_task_progres(upid.to_string()),
+                            Ok(upid) => link.show_task_progress(upid.to_string()),
                             Err(err) => link.show_error(tr!("Error"), err.to_string(), true),
                         }
                     }),
@@ -566,7 +566,7 @@ impl LoadableComponent for PveTreeComp {
                     .on_close(ctx.link().change_view_callback(|_| None))
                     .on_submit({
                         let link = ctx.link().clone();
-                        move |upid: RemoteUpid| link.show_task_progres(upid.to_string())
+                        move |upid: RemoteUpid| link.show_task_progress(upid.to_string())
                     })
                     .into(),
             ),
diff --git a/ui/src/remotes/updates.rs b/ui/src/remotes/updates.rs
index dbb9f700..0aca00e6 100644
--- a/ui/src/remotes/updates.rs
+++ b/ui/src/remotes/updates.rs
@@ -428,7 +428,7 @@ impl LoadableComponent for UpdateTreeComponent {
 
                     match client.refresh_remote_update_summary().await {
                         Ok(upid) => {
-                            link.show_task_progres(upid.to_string());
+                            link.show_task_progress(upid.to_string());
                         }
                         Err(err) => {
                             link.show_error(tr!("Could not refresh update status."), err, false);
diff --git a/ui/src/widget/snapshot_window.rs b/ui/src/widget/snapshot_window.rs
index b14b912f..71b0bc01 100644
--- a/ui/src/widget/snapshot_window.rs
+++ b/ui/src/widget/snapshot_window.rs
@@ -228,7 +228,7 @@ impl PdmSnapshotWindow {
         self.set_task_base_url(
             format!("/{}/remotes/{}/tasks", upid.remote_type(), upid.remote()).into(),
         );
-        ctx.link().show_task_progres(upid.to_string());
+        ctx.link().show_task_progress(upid.to_string());
     }
 
     fn create_input_panel(guest_type: GuestType) -> InputPanel {
-- 
2.47.3





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

* Re: [PATCH datacenter-manager] ui: tree-wide: fix spelling of show-task_progress() function
  2026-06-17 12:41 [PATCH datacenter-manager] ui: tree-wide: fix spelling of show-task_progress() function Shannon Sterz
@ 2026-06-17 13:46 ` Dominik Csapak
  2026-06-18 20:08 ` applied: " Thomas Lamprecht
  1 sibling, 0 replies; 3+ messages in thread
From: Dominik Csapak @ 2026-06-17 13:46 UTC (permalink / raw)
  To: Shannon Sterz, pdm-devel

needed that for testing building with yew-comp from master
so consider this (not applying since it's not bumped yet..)

thanks!

Reviewed-by: Dominik Csapak <d.csapak@proxmox.com>

On 6/17/26 2:40 PM, Shannon Sterz wrote:
> the commit 51a9489dd418 (loadable component: fix show_task_progress
> method name spelling) fixed the name of this function. adapt to it so
> pdm ui will build against yew-comp once this change is bumped.
> 
> Signed-off-by: Shannon Sterz <s.sterz@proxmox.com>
> ---
> 
> Notes:
>      i know the change this is adapting to isn't bumped yet, so applying it
>      before that will break the build. however, i could not find anything
>      like this on list [1] so i thought i'll send a fix for this before
>      it's accidentally missed.
>      
>      [1]: https://lore.proxmox.com/all/?q=show_task_progres
> 
>   ui/src/configuration/subscription_registry.rs | 2 +-
>   ui/src/guests.rs                              | 2 +-
>   ui/src/pve/tree.rs                            | 6 +++---
>   ui/src/remotes/updates.rs                     | 2 +-
>   ui/src/widget/snapshot_window.rs              | 2 +-
>   5 files changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/ui/src/configuration/subscription_registry.rs b/ui/src/configuration/subscription_registry.rs
> index 470789bc..32343c15 100644
> --- a/ui/src/configuration/subscription_registry.rs
> +++ b/ui/src/configuration/subscription_registry.rs
> @@ -885,7 +885,7 @@ impl LoadableComponent for SubscriptionRegistryComp {
>                           Ok(None) => link.change_view(None),
>                           Ok(Some(upid)) => {
>                               link.change_view(None);
> -                            link.show_task_progres(upid);
> +                            link.show_task_progress(upid);
>                           }
>                           Err(err) => link.show_error(tr!("Apply Pending"), err.to_string(), true),
>                       }
> diff --git a/ui/src/guests.rs b/ui/src/guests.rs
> index f11f0de5..ef670a00 100644
> --- a/ui/src/guests.rs
> +++ b/ui/src/guests.rs
> @@ -340,7 +340,7 @@ impl LoadableComponent for GuestPanelComp {
>                   // another remote that ran concurrently cannot point the task
>                   // viewer at the wrong remote
>                   self.set_task_base_url(format!("/pve/remotes/{}/tasks", upid.remote()).into());
> -                ctx.link().show_task_progres(upid.to_string());
> +                ctx.link().show_task_progress(upid.to_string());
>               }
>               Msg::GuestAction(action, key) => {
>                   let Some(entry) = self.store.read().lookup_record(&key).cloned() else {
> diff --git a/ui/src/pve/tree.rs b/ui/src/pve/tree.rs
> index 7a14a05b..0042331c 100644
> --- a/ui/src/pve/tree.rs
> +++ b/ui/src/pve/tree.rs
> @@ -362,7 +362,7 @@ impl LoadableComponent for PveTreeComp {
>                           };
>   
>                           match res {
> -                            Ok(upid) => link.show_task_progres(upid.to_string()),
> +                            Ok(upid) => link.show_task_progress(upid.to_string()),
>                               Err(err) => link.show_error(tr!("Error"), err.to_string(), true),
>                           }
>                       }),
> @@ -386,7 +386,7 @@ impl LoadableComponent for PveTreeComp {
>                           };
>   
>                           match res {
> -                            Ok(upid) => link.show_task_progres(upid.to_string()),
> +                            Ok(upid) => link.show_task_progress(upid.to_string()),
>                               Err(err) => link.show_error(tr!("Error"), err.to_string(), true),
>                           }
>                       }),
> @@ -566,7 +566,7 @@ impl LoadableComponent for PveTreeComp {
>                       .on_close(ctx.link().change_view_callback(|_| None))
>                       .on_submit({
>                           let link = ctx.link().clone();
> -                        move |upid: RemoteUpid| link.show_task_progres(upid.to_string())
> +                        move |upid: RemoteUpid| link.show_task_progress(upid.to_string())
>                       })
>                       .into(),
>               ),
> diff --git a/ui/src/remotes/updates.rs b/ui/src/remotes/updates.rs
> index dbb9f700..0aca00e6 100644
> --- a/ui/src/remotes/updates.rs
> +++ b/ui/src/remotes/updates.rs
> @@ -428,7 +428,7 @@ impl LoadableComponent for UpdateTreeComponent {
>   
>                       match client.refresh_remote_update_summary().await {
>                           Ok(upid) => {
> -                            link.show_task_progres(upid.to_string());
> +                            link.show_task_progress(upid.to_string());
>                           }
>                           Err(err) => {
>                               link.show_error(tr!("Could not refresh update status."), err, false);
> diff --git a/ui/src/widget/snapshot_window.rs b/ui/src/widget/snapshot_window.rs
> index b14b912f..71b0bc01 100644
> --- a/ui/src/widget/snapshot_window.rs
> +++ b/ui/src/widget/snapshot_window.rs
> @@ -228,7 +228,7 @@ impl PdmSnapshotWindow {
>           self.set_task_base_url(
>               format!("/{}/remotes/{}/tasks", upid.remote_type(), upid.remote()).into(),
>           );
> -        ctx.link().show_task_progres(upid.to_string());
> +        ctx.link().show_task_progress(upid.to_string());
>       }
>   
>       fn create_input_panel(guest_type: GuestType) -> InputPanel {





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

* applied: [PATCH datacenter-manager] ui: tree-wide: fix spelling of show-task_progress() function
  2026-06-17 12:41 [PATCH datacenter-manager] ui: tree-wide: fix spelling of show-task_progress() function Shannon Sterz
  2026-06-17 13:46 ` Dominik Csapak
@ 2026-06-18 20:08 ` Thomas Lamprecht
  1 sibling, 0 replies; 3+ messages in thread
From: Thomas Lamprecht @ 2026-06-18 20:08 UTC (permalink / raw)
  To: pdm-devel, Shannon Sterz

On Wed, 17 Jun 2026 14:41:15 +0200, Shannon Sterz wrote:
> the commit 51a9489dd418 (loadable component: fix show_task_progress
> method name spelling) fixed the name of this function. adapt to it so
> pdm ui will build against yew-comp once this change is bumped.

I bumped yew-comp now, so: applied, thanks!

[1/1] ui: tree-wide: fix spelling of show-task_progress() function
      commit: e596b702cb58ad9e8fbd09b78907259ece4d024e




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

end of thread, other threads:[~2026-06-18 20:08 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-06-17 12:41 [PATCH datacenter-manager] ui: tree-wide: fix spelling of show-task_progress() function Shannon Sterz
2026-06-17 13:46 ` Dominik Csapak
2026-06-18 20:08 ` 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