Skip to content

Newsletter settings 846#1063

Open
areldin8 wants to merge 3 commits intohexlet-volunteers:mainfrom
areldin8:newsletter-settings-846
Open

Newsletter settings 846#1063
areldin8 wants to merge 3 commits intohexlet-volunteers:mainfrom
areldin8:newsletter-settings-846

Conversation

@areldin8
Copy link
Copy Markdown

реализация контроллера почтовых уведомлений
Создан NewsletterSettingsController с методами showEditPage (GET) и updateSettings (POST).
Интегрирована поддержка локализации через resolveLocale и FlashPropsService.
Настроено отображение формы через Inertia.js (Account/Newsletters/Edit).
Добавлена обработка ошибок валидации и вывод success-сообщений через Flash-атрибуты.

@areldin8 areldin8 force-pushed the newsletter-settings-846 branch from 12b2b51 to dfbf397 Compare March 25, 2026 16:02
private static final Set<String> SUPPORTED_LOCALES = Set.of("en", "ru");
private static final String DEFAULT_LOCALE = "en";

@GetMapping("/{locale}/account/newsletters/edit")
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Уже слили в main мр, в котором поправили эту историю с локалью - теперь она не передается в качестве параметра

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