From 288cfeb9a8d404a9aea0dd9787a061736f191656 Mon Sep 17 00:00:00 2001
From: Reinout Meliesie <zedfrigg@kernelmaft.com>
Date: Fri, 29 Mar 2024 21:01:36 +0100
Subject: [PATCH] docs: expand comment about dbus client refactor

---
 src/modules/networkmanager.rs | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/modules/networkmanager.rs b/src/modules/networkmanager.rs
index 7b59f18..4786c82 100644
--- a/src/modules/networkmanager.rs
+++ b/src/modules/networkmanager.rs
@@ -54,7 +54,10 @@ impl Module<gtk::Box> for NetworkmanagerModule {
         let tx = context.tx.clone();
 
         spawn(async move {
-            // TODO: Maybe move this into a `client` à la `upower`?
+            /* TODO: This should be moved into a client à la the upower module, however that
+               requires additional refactoring as both would request a PropertyProxy but on
+               different buses. The proper solution will be to rewrite both to use trait-derived
+               proxies. */
             let nm_proxy = {
                 let dbus = zbus::Connection::system().await?;
                 PropertiesProxy::builder(&dbus)