Skip to content

Conversation

@mdanilowicz
Copy link
Contributor

@mdanilowicz mdanilowicz commented Jan 14, 2026

This pull request adds a newsletter subscription confirmation page and introduces localized newsletter-related translations in German, English, and Polish. The main focus is on providing a user interface for newsletter confirmation and ensuring proper internationalization support.

Newsletter Subscription Confirmation Page:

  • Added a new page (newsletter-subscribe.vue) that confirms newsletter subscription status, displays loading and error states, and uses localized messages for user feedback.

Internationalization (i18n) Enhancements:

  • Introduced newsletter.json translation files for German (de-DE), English (en-GB), and Polish (pl-PL) with newsletter-specific strings. [1] [2] [3]
  • Updated the language index files (de-DE.ts, en-GB.ts, pl-PL.ts) to import and merge the new newsletter translations. [1] [2]

closes #1993

@vercel
Copy link

vercel bot commented Jan 14, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
frontends-starter-template-extended Ready Ready Preview, Comment Jan 18, 2026 7:42pm
frontends-vue-starter-template Building Building Preview, Comment Jan 18, 2026 7:42pm
old-frontends-demo Ready Ready Preview, Comment Jan 18, 2026 7:42pm
1 Skipped Deployment
Project Deployment Review Updated (UTC)
shopware-frontends-docs Skipped Skipped Jan 18, 2026 7:42pm

Request Review

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

This pull request adds a newsletter subscription confirmation page with internationalization support for German, English, and Polish languages. The implementation includes proper error handling, loading states, and user feedback messages.

Changes:

  • Added a new Vue page component for newsletter subscription confirmation that processes email confirmation tokens from URL query parameters
  • Introduced localized newsletter translation files with subscription-related messages for three languages (de-DE, en-GB, pl-PL)
  • Updated language index files to integrate the new newsletter translations into the i18n system

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
templates/vue-starter-template/app/pages/newsletter-subscribe.vue New confirmation page that handles newsletter subscription verification via API with loading and error states
templates/vue-starter-template/i18n/de-DE/newsletter.json German translations for newsletter-related UI text and messages
templates/vue-starter-template/i18n/en-GB/newsletter.json English translations for newsletter-related UI text and messages
templates/vue-starter-template/i18n/pl-PL/newsletter.json Polish translations for newsletter-related UI text and messages
templates/vue-starter-template/i18n/de-DE/de-DE.ts Import and merge newsletter translations into German language exports
templates/vue-starter-template/i18n/en-GB/en-GB.ts Import and merge newsletter translations into English language exports
templates/vue-starter-template/i18n/pl-PL/pl-PL.ts Import and merge newsletter translations into Polish language exports

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

@patzick patzick left a comment

Choose a reason for hiding this comment

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

We need action button for the user for what to do next and we need some space before footer as this feels very tight (as on screenshot)

@patzick patzick merged commit 60d15e2 into main Jan 19, 2026
11 checks passed
@patzick patzick deleted the feat/GH-1993 branch January 19, 2026 10:17
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.

Layers / Pages / Newsletter Confirmation Page

4 participants