Prevent campaign details being edited during send (#620)

parcelvoy · Jan 25, 2025 · 1527e01 · 1527e01
1 parent c24e37c
commit 1527e01
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/apps/platform/src/queue/RedisQueueProvider.ts b/apps/platform/src/queue/RedisQueueProvider.ts
@@ -24,7 +24,7 @@ export default class RedisQueueProvider implements QueueProvider {
 
     constructor({ concurrency, ...config }: RedisQueueConfig, queue: Queue) {
         this.queue = queue
-        this.concurrency = concurrency
+        this.concurrency = concurrency ?? 10
         this.redis = DefaultRedis(config, {
             maxRetriesPerRequest: null,
         })

diff --git a/apps/ui/src/views/campaign/CampaignOverview.tsx b/apps/ui/src/views/campaign/CampaignOverview.tsx
@@ -30,6 +30,8 @@ export default function CampaignOverview() {
         )?.reduce((prev, curr) => prev ? [prev, ', ', curr] : curr, '') ?? '&#8211;'
     }
 
+    const canEdit = campaign.type === 'trigger' || campaign.state === 'draft' || campaign.state === 'aborted'
+
     const extra = campaign.channel === 'text'
         ? '"phone": "+12345678900",'
         : campaign.channel === 'push'
@@ -57,7 +59,7 @@ export default function CampaignOverview() {
                 title={t('details')}
                 size="h3"
                 actions={
-                    <Button
+                    canEdit && <Button
                         size="small"
                         variant="secondary"
                         onClick={() => setIsEditOpen(true)}

diff --git a/docs/docs/advanced/config.md b/docs/docs/advanced/config.md
@@ -18,7 +18,7 @@ Find below a list of all environment variables that can be set at launch to conf
 | DB_DATABASE | string | true |
 
 ### Queue
-| key | type | required |
+| key | type | required | notes
 |--|--|--|
 | QUEUE_DRIVER | 'sqs' or 'redis' or 'memory' | true |
 | AWS_SQS_QUEUE_URL | string | If driver is SQS |
@@ -28,6 +28,7 @@ Find below a list of all environment variables that can be set at launch to conf
 | REDIS_HOST | string | If driver is Redis |
 | REDIS_PORT | string | If driver is Redis |
 | REDIS_TLS | boolean | false |
+| REDIS_CONCURRENCY | number | false | number of concurrent jobs to run
 
 
 ### Redis