🔀 Merge pull request #28 from davep/tweak-move

Improve the interface of Subscriptions.move

Author: davep

ChangeLog.md

@@ -1,5 +1,13 @@
 # OldAS ChangeLog
 
+## Unreleased
+
+**Released: WiP**
+
+- Updated `Subscriptions.move` so that any form of "empty" for the
+  `target_folder` is seen as a "remove folder" operation.
+  ([#28](https://github.com/davep/oldas/pull/28))
+
 ## v0.5.0
 
 **Released: 2026-01-24**

src/oldas/subscriptions.py

@@ -272,12 +272,15 @@ async def move(
             [`True`][True] if the move call worked, [`False`][False] if not.
 
         Note:
-            If `target_folder` is omitted, the subscription will be moved to
-            the top-level default folder.
+            If `target_folder` is omitted, is [`None`][None], or is an empty
+            [string][str], the subscription will be moved to the top-level
+            default folder.
         """
+        if isinstance(target_folder, str):
+            target_folder = target_folder.strip()
         operation = (
             {"r": "remove"}
-            if target_folder is None
+            if not target_folder
             else {"a": Folders.full_id(target_folder)}
         )
         return await session.post_ok(