Skip to content

Add email on release approval#3768

Merged
GB907762 merged 4 commits into
mainfrom
feature/email-on-release
Jun 24, 2026
Merged

Add email on release approval#3768
GB907762 merged 4 commits into
mainfrom
feature/email-on-release

Conversation

@GB907762

Copy link
Copy Markdown
Member

Release email is sent to everyone who has a system role on a model (not review roles).
Release email is sent to everyone on a model with an approved access request.

Release email is sent to everyone who has a system role on a model (not review roles). Release email is sent to everyone on a model with an approved access request.
@github-actions github-actions Bot added enhancement New feature or request javascript Pull requests that update Javascript code tests Improvements or additions to unit tests and / or end to end tests backend Changes affecting server-side logic, APIs, data processing, or internal services. labels Jun 23, 2026
Comment on lines +17 to +38
registerPath(
{
method: 'get',
path: '/api/v3/review/{reviewId}/responses/latest',
tags: ['response'],
description: 'Get the latest response for a review',
schema: getLatestResponseSchema,
responses: {
200: {
description: 'A review response.',
content: {
'application/json': {
schema: z.object({
response: responseInterfaceSchema,
}),
},
},
},
},
},
})
'v3',
)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While debugging. I noticed this route was on the v2 swagger docs.

Comment thread backend/src/services/smtp/smtp.ts Outdated
Comment thread backend/src/services/smtp/smtp.ts Outdated
Comment thread backend/test/services/smtp/smtp.spec.ts Outdated
Comment thread backend/test/services/smtp/smtp.spec.ts Outdated
Comment thread backend/test/services/response.spec.ts Outdated
Comment thread backend/src/services/response.ts Outdated
@GB907762 GB907762 merged commit 1b5f328 into main Jun 24, 2026
24 checks passed
@GB907762 GB907762 deleted the feature/email-on-release branch June 24, 2026 10:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend Changes affecting server-side logic, APIs, data processing, or internal services. enhancement New feature or request javascript Pull requests that update Javascript code tests Improvements or additions to unit tests and / or end to end tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants