From 817e7d2e37d982250ed871ce256d9afd037756a2 Mon Sep 17 00:00:00 2001
From: arabcoders <admin@arabcoders.org>
Date: Tue, 8 Apr 2025 23:30:08 +0300
Subject: [PATCH 1/6] minor ui change

---
 ui/layouts/default.vue | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/ui/layouts/default.vue b/ui/layouts/default.vue
index 2b0c5282..6d7675a1 100644
--- a/ui/layouts/default.vue
+++ b/ui/layouts/default.vue
@@ -205,6 +205,10 @@ onMounted(async () => {
 
 watch(selectedTheme, value => {
   try {
+    if ('auto' === value) {
+      applyPreferredColorScheme(window.matchMedia('(prefers-color-scheme: dark)').matches ? 'dark' : 'light')
+      return
+    }
     applyPreferredColorScheme(value)
   } catch (e) { }
 })

From 88163cad42d240d06d2658704bb688b9debc7844 Mon Sep 17 00:00:00 2001
From: arabcoders <admin@arabcoders.org>
Date: Wed, 9 Apr 2025 17:35:11 +0300
Subject: [PATCH 2/6] make it possible to change notification data field key
 name

---
 app/library/Notifications.py       | 18 ++++++++++++++++--
 ui/components/NotificationForm.vue | 29 ++++++++++++++++++++++++++---
 ui/pages/notifications.vue         |  4 ++--
 3 files changed, 44 insertions(+), 7 deletions(-)

diff --git a/app/library/Notifications.py b/app/library/Notifications.py
index a62e1bf3..91923e44 100644
--- a/app/library/Notifications.py
+++ b/app/library/Notifications.py
@@ -46,12 +46,14 @@ class TargetRequest:
     method: str
     url: str
     headers: list[TargetRequestHeader] = field(default_factory=list)
+    data_key: str = "data"
 
     def serialize(self) -> dict:
         return {
             "type": self.type,
             "method": self.method,
             "url": self.url,
+            "data_key": self.data_key,
             "headers": [h.serialize() for h in self.headers],
         }
 
@@ -247,6 +249,7 @@ class Notification(metaclass=Singleton):
                 type=target.get("request", {}).get("type", "json"),
                 method=target.get("request", {}).get("method", "POST"),
                 url=target.get("request", {}).get("url"),
+                data_key=target.get("request", {}).get("data_key", "data"),
                 headers=[
                     TargetRequestHeader(
                         key=str(h.get("key", "")).strip(),
@@ -288,6 +291,10 @@ class Notification(metaclass=Singleton):
             msg = "Invalid notification target. No URL found."
             raise ValueError(msg)
 
+        if "data_key" not in target["request"]:
+            msg = "Invalid notification target. No data_key found."
+            raise ValueError(msg)
+
         if "method" in target["request"] and target["request"]["method"].upper() not in ["POST", "PUT"]:
             msg = "Invalid notification target. Invalid method found."
             raise ValueError(msg)
@@ -363,10 +370,17 @@ class Notification(metaclass=Singleton):
                 for h in target.request.headers:
                     reqBody["headers"][h.key] = h.value
 
-            reqBody["json" if "json" == target.request.type.lower() else "data"] = self._deep_unpack(ev.serialize())
+            body_key = "json" if "json" == target.request.type.lower() else "data"
+            reqBody[body_key] = self._deep_unpack(ev.serialize())
+
+            if "data" != target.request.data_key:
+                reqBody[body_key][target.request.data_key] = reqBody[body_key]["data"]
+                reqBody[body_key].pop("data", None)
 
             if "form" == target.request.type.lower():
-                reqBody["data"]["data"] = self._encoder.encode(reqBody["data"]["data"])
+                reqBody[body_key][target.request.data_key] = self._encoder.encode(
+                    reqBody[body_key][target.request.data_key]
+                )
 
             response = await self._client.request(**reqBody)
 
diff --git a/ui/components/NotificationForm.vue b/ui/components/NotificationForm.vue
index 112e54d4..87130c3f 100644
--- a/ui/components/NotificationForm.vue
+++ b/ui/components/NotificationForm.vue
@@ -138,7 +138,7 @@
                 </div>
               </div>
 
-              <div class="column is-12 is-clearfix">
+              <div class="column is-6-tablet is-12-mobile">
                 <div class="field">
                   <label class="label is-inline" for="on">
                     Select Events
@@ -167,6 +167,26 @@
                 </div>
               </div>
 
+              <div class="column is-6-tablet is-12-mobile">
+                <div class="field">
+                  <label class="label is-inline" for="data_key">
+                    Data field
+                  </label>
+                  <div class="control has-icons-left">
+                    <input type="text" class="input" id="data_key" v-model="form.request.data_key"
+                      :disabled="addInProgress" required>
+                    <span class="icon is-small is-left"><i class="fa-solid fa-key" /></span>
+                  </div>
+                  <span class="help">
+                    <span class="icon"><i class="fa-solid fa-info" /></span>
+                    <span>
+                      The field name to use when sending the notification. This is used to identify the data in the
+                      request. The default is <code>data</code>.
+                    </span>
+                  </span>
+                </div>
+              </div>
+
               <div class="column is-12">
                 <div class="field">
                   <label class="label is-inline is-unselectable">
@@ -277,7 +297,7 @@ const showImport = useStorage('showImport', false);
 const import_string = ref('');
 
 const checkInfo = async () => {
-  const required = ['name', 'request.url', 'request.method', 'request.type'];
+  const required = ['name', 'request.url', 'request.method', 'request.type', 'request.data_key'];
   for (const key of required) {
     if (key.includes('.')) {
       const [parent, child] = key.split('.');
@@ -356,6 +376,10 @@ const importItem = async () => {
       form.request = item.request
     }
 
+    if (item.data_key) {
+      form.data_key = item.data_key
+    }
+
     if (item.on) {
       form.on = item.on
     }
@@ -366,5 +390,4 @@ const importItem = async () => {
     toast.error(`Failed to import task. ${e.message}`)
   }
 }
-
 </script>
diff --git a/ui/pages/notifications.vue b/ui/pages/notifications.vue
index fcc65cbd..e31905ac 100644
--- a/ui/pages/notifications.vue
+++ b/ui/pages/notifications.vue
@@ -126,8 +126,8 @@ div.is-centered {
           </li>
           <li>
             When you set the request type as <code>Form</code>, the event data will be JSON encoded and sent as the
-            and sent as <code>...&data=json_string</code>, only the <code>data</code> field will be JSON encoded. The
-            other keys <code>id</code>, <code>event</code> and <code>created_at</code> will be sent as they are.
+            and sent as <code>...&data_key=json_string</code>, only the <code>data</code> field will be JSON encoded.
+            The other keys <code>id</code>, <code>event</code> and <code>created_at</code> will be sent as they are.
           </li>
           <li>We also send two special headers <code>X-Event-ID</code> and <code>X-Event</code> with the request.</li>
         </ul>

From c2fa0666692cd61b2b3e3d145306f76d2bb5fc35 Mon Sep 17 00:00:00 2001
From: arabcoders <admin@arabcoders.org>
Date: Wed, 9 Apr 2025 19:36:49 +0300
Subject: [PATCH 3/6] set default data_key for notification targets if not
 provided

---
 app/library/Notifications.py | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/app/library/Notifications.py b/app/library/Notifications.py
index 91923e44..a1a7f6f4 100644
--- a/app/library/Notifications.py
+++ b/app/library/Notifications.py
@@ -292,8 +292,7 @@ class Notification(metaclass=Singleton):
             raise ValueError(msg)
 
         if "data_key" not in target["request"]:
-            msg = "Invalid notification target. No data_key found."
-            raise ValueError(msg)
+            target["request"]["data_key"] = "data"
 
         if "method" in target["request"] and target["request"]["method"].upper() not in ["POST", "PUT"]:
             msg = "Invalid notification target. Invalid method found."

From ee2fd3f398ac15045c0bf8bebca4dcf518e92cc4 Mon Sep 17 00:00:00 2001
From: arabcoders <admin@arabcoders.org>
Date: Wed, 9 Apr 2025 19:41:54 +0300
Subject: [PATCH 4/6] Only show notification target, id or url incase of error
 loading the target.

---
 app/library/Notifications.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/app/library/Notifications.py b/app/library/Notifications.py
index a1a7f6f4..cdfb27d3 100644
--- a/app/library/Notifications.py
+++ b/app/library/Notifications.py
@@ -215,7 +215,8 @@ class Notification(metaclass=Singleton):
                 try:
                     Notification.validate(target)
                 except ValueError as e:
-                    LOG.error(f"Invalid notification target '{target}'. '{e!s}'")
+                    name = target.get("name") or target.get("id") or target.get("request", {}).get("url") or "unknown"
+                    LOG.error(f"Invalid notification target '{name}'. '{e!s}'")
                     continue
 
                 target = self.make_target(target)

From 4a21592e12447f2be11f1472d31861907ab0ba41 Mon Sep 17 00:00:00 2001
From: arabcoders <admin@arabcoders.org>
Date: Thu, 10 Apr 2025 00:59:52 +0300
Subject: [PATCH 5/6] raise an error if item was using invalid preset.

---
 app/library/ItemDTO.py |  6 ++++++
 app/library/Presets.py | 13 +++++++++++++
 2 files changed, 19 insertions(+)

diff --git a/app/library/ItemDTO.py b/app/library/ItemDTO.py
index e9fc570e..2619675a 100644
--- a/app/library/ItemDTO.py
+++ b/app/library/ItemDTO.py
@@ -134,6 +134,12 @@ class Item:
 
         preset = item.get("preset")
         if preset and isinstance(preset, str) and preset != Item._default_preset():
+            from .Presets import Presets
+
+            if not Presets.get_instance().has(preset):
+                msg = f"Preset '{preset}' does not exist."
+                raise ValueError(msg)
+
             data["preset"] = preset
 
         if item.get("folder") and isinstance(item.get("folder"), str):
diff --git a/app/library/Presets.py b/app/library/Presets.py
index cfec885b..a123e7c3 100644
--- a/app/library/Presets.py
+++ b/app/library/Presets.py
@@ -266,6 +266,19 @@ class Presets(metaclass=Singleton):
 
         return self
 
+    def has(self, id_or_name: str) -> bool:
+        """
+        Check if the preset exists by id or name.
+
+        Args:
+            id_or_name (str): The id or name of the preset.
+
+        Returns:
+            bool: True if the preset exists, False otherwise.
+
+        """
+        return self.get(id_or_name) is not None
+
     def get(self, id_or_name: str) -> Preset | None:
         """
         Get the preset by id or name.

From 17dd007ff03987cbd4445d026a6ef5e611a10cf7 Mon Sep 17 00:00:00 2001
From: arabcoders <admin@arabcoders.org>
Date: Thu, 10 Apr 2025 01:32:32 +0300
Subject: [PATCH 6/6] Show that a external dl is being used if we are stuck in
 preparing stage as yt-dlp doesn't trigger progress_hook

---
 app/library/DownloadQueue.py | 22 ++++++++++++++++++++--
 ui/components/Queue.vue      |  6 +++++-
 2 files changed, 25 insertions(+), 3 deletions(-)

diff --git a/app/library/DownloadQueue.py b/app/library/DownloadQueue.py
index df84ba68..07e92b22 100644
--- a/app/library/DownloadQueue.py
+++ b/app/library/DownloadQueue.py
@@ -184,8 +184,8 @@ class DownloadQueue(metaclass=Singleton):
         """
         if item.has_extras():
             for key in item.extras.copy():
-                if not str(key).startswith("playlist"):
-                    item.extras.pop(key, None)
+                if not self.keep_extra_key(key):
+                    item.extras.pop(key)
 
         if not entry:
             return {"status": "error", "msg": "Invalid/empty data was given."}
@@ -395,6 +395,10 @@ class DownloadQueue(metaclass=Singleton):
                 .get_all(),
             }
 
+            if yt_conf.get("external_downloader"):
+                LOG.warning(f"Using external downloader '{yt_conf.get('external_downloader')}' for '{item.url}'.")
+                item.extras.update({"external_downloader": True})
+
             downloaded, id_dict = self._is_downloaded(file=yt_conf.get("download_archive", None), url=item.url)
             if downloaded is True and id_dict:
                 message = f"This url with ID '{id_dict.get('id')}' has been downloaded already and recorded in archive."
@@ -715,3 +719,17 @@ class DownloadQueue(metaclass=Singleton):
             return False, None
 
         return is_downloaded(file, url)
+
+    def keep_extra_key(self, key: str) -> bool:
+        """
+        Check if the extra key should be kept.
+
+        Args:
+            key (str): The extra key to check.
+
+        Returns:
+            bool: True if the extra key should be kept, False otherwise.
+
+        """
+        keys = ("playlist", "external_downloader")
+        return any(key == k or key.startswith(k) for k in keys)
diff --git a/ui/components/Queue.vue b/ui/components/Queue.vue
index 03f7cd1c..066c477a 100644
--- a/ui/components/Queue.vue
+++ b/ui/components/Queue.vue
@@ -301,6 +301,10 @@ const setStatus = item => {
     return 'Live Streaming';
   }
 
+  if ('preparing' === item.status) {
+    return ag(item, 'extras.external_downloader') ? 'External DL' : 'Preparing..';
+  }
+
   if (!item.status) {
     return 'Unknown..';
   }
@@ -372,7 +376,7 @@ const updateProgress = (item) => {
   }
 
   if ('preparing' === item.status) {
-    return 'Preparing';
+    return ag(item, 'extras.external_downloader') ? 'External downloader.' : 'Preparing';
   }
 
   if (null != item.status) {