Skip to content

Conversation

@younocode
Copy link
Contributor

No description provided.

@younocode younocode force-pushed the feat/email-unsubscription branch 3 times, most recently from 31e7625 to e9d8467 Compare October 15, 2025 14:56
@younocode younocode requested a review from Copilot October 15, 2025 15:05
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Introduce email unsubscription capabilities across the OpenAPI client, UI strings, and backend routing.

  • Add OpenAPI routes and client functions for unsubscribe get/update, listing, export, and import.
  • Extend i18n strings with unsubscribe-related texts.
  • Add Next.js SSR catch-all route for unsubscribe pages and introduce mail encryption config.

Reviewed Changes

Copilot reviewed 20 out of 20 changed files in this pull request and generated 11 comments.

Show a summary per file
File Description
packages/openapi/src/unsubscribe/update.ts Adds POST route and client to update subscription status by token.
packages/openapi/src/unsubscribe/index.ts Re-exports unsubscribe module APIs.
packages/openapi/src/unsubscribe/import-list.ts Adds route and client for importing unsubscribe lists.
packages/openapi/src/unsubscribe/get.ts Adds GET route and client to fetch unsubscribe info by token; introduces related types.
packages/openapi/src/unsubscribe/get-list.ts Adds GET route and client to fetch paginated unsubscribe list.
packages/openapi/src/unsubscribe/export-list.ts Adds route and client for exporting unsubscribe list.
packages/openapi/src/mail/types.ts Adds AutomationSendEmailAction mail type.
packages/openapi/src/index.ts Exposes unsubscribe APIs from root index.
packages/common-i18n/src/locales/*/common.json Adds unsubscribe-related UI texts and labels in multiple locales.
apps/nestjs-backend/src/features/next/next.controller.ts Adds Next.js SSR route pattern for unsubscribe pages.
apps/nestjs-backend/src/configs/mail.config.ts Adds mail encryption config (algorithm, key, IV, encoding).

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@younocode younocode force-pushed the feat/email-unsubscription branch from 2ddbb4c to 055f75d Compare October 17, 2025 06:41
@coveralls
Copy link
Collaborator

coveralls commented Oct 20, 2025

Pull Request Test Coverage Report for Build 18739098653

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 6 of 7 (85.71%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.004%) to 75.211%

Changes Missing Coverage Covered Lines Changed/Added Lines %
apps/nestjs-backend/src/features/next/next.controller.ts 0 1 0.0%
Totals Coverage Status
Change from base Build 18738246368: 0.004%
Covered Lines: 50358
Relevant Lines: 66956

💛 - Coveralls

@younocode younocode force-pushed the feat/email-unsubscription branch 4 times, most recently from 2363167 to aa588d1 Compare October 22, 2025 07:14
@younocode younocode force-pushed the feat/email-unsubscription branch from 475f5f2 to 1203315 Compare October 23, 2025 06:03
@younocode younocode requested a review from boris-w October 23, 2025 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants