Skip to content

Commit

Permalink
Enable slack deployment notifications
Browse files Browse the repository at this point in the history
This differs from the `notify_slack` setting, which is more like
`alert_slack_on_failed_deployments`.

I'll maybe rename this separately...

Related:
communitiesuk/funding-service-design-workflows#265
  • Loading branch information
samuelhwilliams committed Feb 4, 2025
1 parent 4c0a6ac commit 576d053
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 2 deletions.
2 changes: 2 additions & 0 deletions .github/workflows/copilot_deploy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -475,10 +475,12 @@ jobs:
AWS_ACCOUNT: ${{ secrets.AWS_ACCOUNT }}
SLACK_BOT_TOKEN: ${{ secrets.SLACK_BOT_TOKEN }}
SLACK_NOTIFICATION_CHANNEL_ID: ${{ secrets.SLACK_NOTIFICATION_CHANNEL_ID }}
SLACK_DEPLOYMENTS_CHANNEL_ID: ${{ secrets.SLACK_DEPLOYMENTS_CHANNEL_ID }}
with:
environment: prod
app_name: form-runner-adapter
run_db_migrations: false
image_location: ghcr.io/communitiesuk/funding-service-design-form-runner-adapter:sha-${{ github.sha }}
notify_slack: true
notify_slack_on_deployment: true

Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ export const putFormWithIdRouteRegister: ServerRoute = {
await persistenceService.uploadConfiguration(
`${id}`,
JSON.stringify(value)
);
);e
await publish(id, value);
return h.response({ok: true}).code(204);
} catch (err) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ export class RegisterFormPublishApi implements RegisterApi {
if (!previewMode || previewMode==="false") {
request.logger.error(
[`POST /publish`, "previewModeError"],
disabledRouteDetailString
disabledRouteDetailStringe
);
throw Boom.forbidden("Publishing is disabled");
}
Expand Down

0 comments on commit 576d053

Please sign in to comment.