Skip to content

Oauth#81

Open
kapitulin24 wants to merge 5 commits into
devfrom
refactor/oauth
Open

Oauth#81
kapitulin24 wants to merge 5 commits into
devfrom
refactor/oauth

Conversation

@kapitulin24

Copy link
Copy Markdown
Contributor

Что сделано

  • Перенесена обработка OAuth в app/(auth)/oauth/route.ts, удалена страница app/(auth)/oauth/page.tsx.
  • Обновлены провайдеры и типы OAuth (oauth-providers, схемы и типы auth), чтобы выровнять контракт.
  • Упрощены OAuth-кнопки: удален OAuthButton, логика разложена на OAuthLoginButtonsContent и OAuthSeparatorContent.
  • Обновлены SigninForm и AccountsSection для работы с новым OAuth-потоком.
  • Удалены устаревшие UI-части (OAuthPage и связанный экспорт).

Зачем

Сделать OAuth-поток более предсказуемым и поддерживаемым: отделить обработку роутинга от UI, уменьшить связность компонентов и упростить дальнейшее расширение провайдеров.

Проверка

  • Логин через Google/GitHub/VK/Yandex проходит успешно
  • После OAuth-колбэка выполняется корректный редирект
  • Вход через форму signin работает без регрессий
  • Подключение/отключение OAuth-провайдера в профиле работает корректно
  • Линтер и типизация проходят без ошибок

@kapitulin24 kapitulin24 requested review from AlexandrNel and soorq June 19, 2026 15:40
@github-actions github-actions Bot added features User scenarios and sliced features domain Business entities and models views Pages, widgets and layouts config labels Jun 19, 2026
Comment thread app/(auth)/oauth/route.ts Outdated
Comment thread src/entities/auth/config/oauth-providers.ts
Comment thread src/pages/auth/signin/ui/SigninForm.tsx Outdated
soorq
soorq previously approved these changes Jun 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

config domain Business entities and models features User scenarios and sliced features views Pages, widgets and layouts

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants