From e67c7e73223d2299bbeccdc6e69d76a9de1a06b3 Mon Sep 17 00:00:00 2001
From: Steven Nguyen <stnguyen90@users.noreply.github.com>
Date: Wed, 21 Feb 2024 18:44:30 -0800
Subject: [PATCH] fix(messaging): fix user targets modal title and description

The title and description should be different depending on whether you're adding a target to a message or a target to a topic.
---
 .../project-[project]/messaging/actions.svelte   |  6 +++++-
 .../topic-[topic]/subscribers/+page.svelte       | 16 +++++++++++++++-
 .../messaging/userTargetsModal.svelte            | 11 +++--------
 3 files changed, 23 insertions(+), 10 deletions(-)

diff --git a/src/routes/console/project-[project]/messaging/actions.svelte b/src/routes/console/project-[project]/messaging/actions.svelte
index 110779982..f52264348 100644
--- a/src/routes/console/project-[project]/messaging/actions.svelte
+++ b/src/routes/console/project-[project]/messaging/actions.svelte
@@ -36,10 +36,14 @@
         dispatch('addTopics', e.detail);
     }} />
 <UserTargetsModal
+    title="Select targets"
     {providerType}
     bind:show={showUserTargets}
     bind:targetsById={$targetsById}
     on:update={(e) => {
         showUserTargets = false;
         dispatch('addTargets', e.detail);
-    }} />
+    }}>
+    <svelte:fragment slot="description"
+        >Select existing targets to which you want to send this message.</svelte:fragment>
+</UserTargetsModal>
diff --git a/src/routes/console/project-[project]/messaging/topics/topic-[topic]/subscribers/+page.svelte b/src/routes/console/project-[project]/messaging/topics/topic-[topic]/subscribers/+page.svelte
index 27219e4e0..08d6fd58a 100644
--- a/src/routes/console/project-[project]/messaging/topics/topic-[topic]/subscribers/+page.svelte
+++ b/src/routes/console/project-[project]/messaging/topics/topic-[topic]/subscribers/+page.svelte
@@ -146,4 +146,18 @@
     {/if}
 </Container>
 
-<UserTargetsModal bind:show={showAdd} bind:targetsById={$targetsById} on:update={addTargets} />
+<UserTargetsModal
+    title="Select subscribers"
+    bind:show={showAdd}
+    bind:targetsById={$targetsById}
+    on:update={addTargets}>
+    <svelte:fragment slot="description">
+        <p class="text">
+            Add subscribers to this topic by selecting the targets for directing messages. <a
+                href="https://appwrite.io/docs/products/messaging/topics"
+                target="_blank"
+                rel="noopener noreferrer"
+                class="link">Learn more about subscribers.</a>
+        </p>
+    </svelte:fragment>
+</UserTargetsModal>
diff --git a/src/routes/console/project-[project]/messaging/userTargetsModal.svelte b/src/routes/console/project-[project]/messaging/userTargetsModal.svelte
index 827b3fe15..7552144aa 100644
--- a/src/routes/console/project-[project]/messaging/userTargetsModal.svelte
+++ b/src/routes/console/project-[project]/messaging/userTargetsModal.svelte
@@ -12,10 +12,10 @@
     import { createEventDispatcher } from 'svelte';
     import ProviderType from './providerType.svelte';
 
+    export let title: string;
     export let show: boolean;
     export let targetsById: Record<string, Models.Target>;
     export let providerType: MessagingProviderType = null;
-    export let title = 'Select subscribers';
 
     const dispatch = createEventDispatcher();
 
@@ -118,13 +118,8 @@
 </script>
 
 <Modal {title} bind:show onSubmit={submit} on:close={reset} size="big" headerDivider={false}>
-    <p class="text">
-        Add subscribers to this topic by selecting the targets for directing messages. <a
-            href="https://appwrite.io/docs/products/messaging/topics"
-            target="_blank"
-            rel="noopener noreferrer"
-            class="link">Learn more about subscribers.</a>
-    </p>
+    <slot name="description" />
+
     <InputSearch
         autofocus
         disabled={totalResults === 0 && !search}