diff --git a/modules/services/samba-share.nix b/modules/services/samba-share.nix
index 8ad5ed0..68463ea 100644
--- a/modules/services/samba-share.nix
+++ b/modules/services/samba-share.nix
@@ -44,7 +44,7 @@ in {
           name = name;
           value = {
             browsable = "yes";
-            comment = "read only share {name}";
+            comment = "read only share ${name}";
             path = path;
             "read only" = "yes";
             "guest ok" = "yes";
@@ -61,8 +61,9 @@ in {
       };
 
     })
-    (mkIf config.services.syncthing.enable {
-      users.groups.syncthing.members = [ "smbguest" ];
+    # todo : maybe better to have a parameter for this
+    (mkIf config.test.services.syncthing.enable {
+      users.groups."syncthing".members = [ "smbguest" ];
     })
   ];
 }