Skip to content

Commit 9348d99

Browse files
Fix: [AEA-0000] - Temporarily Remove Document Notification (#396)
## Summary Removes the slack notification when documents are updated. With a AWS Simple Queue Service (SQS), only one Lambda can successfully pick up a message, meaning EITHER this will run OR the sync will run - which is counter-intuitive. This will be removed while [D298](https://nhsd-confluence.digital.nhs.uk/spaces/APIMC/pages/1305328394/D298+Knowledge-base+Document+Change+Notifications) is being considered.
1 parent 29fa06c commit 9348d99

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

packages/cdk/resources/S3LambdaNotification.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,9 @@ export class S3LambdaNotification extends Construct {
2727
queueName: queueName,
2828
batchDelay: 100,
2929
functions: [
30-
props.functions.notifyS3UploadFunction,
30+
// Temporarily only trigger sync, as only one can run at once on SQS
31+
// - if notifications is successful, sync won't run
32+
// props.functions.notifyS3UploadFunction,
3133
props.functions.syncKnowledgeBaseFunction
3234
]
3335
})

0 commit comments

Comments
 (0)