Skip to content

Commit f106b85

Browse files
feat: remove newsletter subscribe on newsletter sub (#475)
1 parent 51a339c commit f106b85

File tree

2 files changed

+0
-35
lines changed

2 files changed

+0
-35
lines changed

libs/blog-bff/newsletter/src/lib/api.ts

Lines changed: 0 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -43,20 +43,16 @@ app.post('/subscribe', async (c) => {
4343
const { BREVO_API_KEY, BREVO_API_URL } = env(c);
4444

4545
let listIds: number[];
46-
let templateId: number;
4746

4847
if (lang === 'pl') {
4948
listIds = [NewsletterList.PL, NewsletterList.PLNew];
50-
templateId = NewsletterTemplate.PL;
5149
} else {
5250
listIds = [NewsletterList.EN];
53-
templateId = NewsletterTemplate.EN;
5451
}
5552

5653
try {
5754
const parsedEmail = v.parse(EmailSchema, newSubscriber);
5855
const client = new NewsletterClient(BREVO_API_URL, BREVO_API_KEY);
59-
let sendTemplate = true;
6056

6157
try {
6258
const existingContact = await client.getContact(parsedEmail);
@@ -75,9 +71,6 @@ app.post('/subscribe', async (c) => {
7571
listIds: mergedListIds,
7672
});
7773
}
78-
79-
// Contact is already on the list, we should not send a welcoming template
80-
sendTemplate = !alreadySubscribed;
8174
} catch (err) {
8275
if (
8376
typeof err === 'object' &&
@@ -94,23 +87,6 @@ app.post('/subscribe', async (c) => {
9487
}
9588
}
9689

97-
if (sendTemplate) {
98-
const template = await client.getTemplate(templateId);
99-
100-
await client.sendEmail({
101-
sender: {
102-
id: template.sender.id,
103-
},
104-
subject: template.subject,
105-
htmlContent: template.htmlContent,
106-
to: [
107-
{
108-
email: parsedEmail,
109-
},
110-
],
111-
});
112-
}
113-
11490
return c.json({ success: true }, 200);
11591
} catch (e) {
11692
if (e instanceof v.ValiError) {

libs/blog-bff/newsletter/src/lib/newsletter-client.ts

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -6,17 +6,6 @@ export class NewsletterClient {
66
private _apiKey: string,
77
) {}
88

9-
getTemplate(templateId: number): Promise<Template> {
10-
return this.request<Template>(`smtp/templates/${templateId}`);
11-
}
12-
13-
sendEmail(dto: SendEmailDto): Promise<void> {
14-
return this.request<void>(`smtp/email`, {
15-
method: 'POST',
16-
body: dto,
17-
});
18-
}
19-
209
createContact(contact: NewContactDto): Promise<void> {
2110
return this.request(`contacts`, {
2211
method: 'POST',

0 commit comments

Comments
 (0)