Skip to content

Story/RSS-CORE-00_fixPaths - #25

Merged
dilmun1101 merged 1 commit into
feature/authfrom
story/RSS-CORE-00_fixPaths
Jul 7, 2026
Merged

dilmun1101 merged 1 commit into
feature/authfrom
story/RSS-CORE-00_fixPaths

Conversation

@dilmun1101

@dilmun1101 dilmun1101 commented Jul 7, 2026 •

Copy link
Copy Markdown
Collaborator

📋 Trello Task ID

  • RSS-SE-## (Task ID / Trello Card)
  • No ID (Global fix or chore)

⚡️ Summary

Change paths to constants

🛠 Type of change

  • feat (New feature)
  • fix (Bug fix)
  • refactor (Code improvement / Refactoring without changing logic)
  • style (Formatting, CSS)
  • docs (Documentation)
  • chore (Configs, Build)
  • test (Testing)

Summary by CodeRabbit

  • Bug Fixes
    • Улучшена навигация между входом и регистрацией: ссылки теперь ведут по единым маршрутам.
    • Исправлены редиректы для защищённых страниц и страниц авторизации, чтобы пользователи попадали на корректные экраны.
    • При попытке открыть недоступный раздел без входа теперь используется более предсказуемый переход на главную страницу.

@vercel

vercel Bot commented Jul 7, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
swagger-editor-app Ready Ready Preview, Comment Jul 7, 2026 3:26pm

@coderabbitai

coderabbitai Bot commented Jul 7, 2026 •

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

Захардкоженные строковые пути в компоненте AuthForm и middleware Supabase заменены на значения из константы ROUTES. Обновлены href ссылки переключения логин/регистрация, массивы защищённых и auth-путей, а также целевые пути редиректов.

Changes

Использование ROUTES вместо хардкода путей

Layer / File(s) Summary
Ссылки переключения режима в AuthForm
src/features/auth-form/ui/auth-form.tsx
Добавлен импорт ROUTES, href ссылки переключения логин/регистрация теперь использует ROUTES.LOGIN и ROUTES.REGISTER вместо строк.
Редиректы и защищённые пути в middleware
src/shared/api/supabase/middleware.ts
Импортирован ROUTES, массивы PROTECTED_PATHS/AUTH_PATHS заменены значениями из ROUTES, а целевые пути редиректов заменены на ROUTES.HOME вместо '/'.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Suggested reviewers: lexarudak, FierceSloth

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed Заголовок связан с изменением путей и частично отражает суть PR, хотя выглядит как технический идентификатор.
Description check ✅ Passed Описание в основном заполнено по шаблону: указан ID, краткое резюме и тип изменения.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch story/RSS-CORE-00_fixPaths

Comment @coderabbitai help to get the list of available commands.

@dilmun1101 dilmun1101 changed the title refactor: change paths Story/RSS-CORE-00_fixPaths Jul 7, 2026
@dilmun1101
dilmun1101 merged commit 59e009a into feature/auth Jul 7, 2026
4 checks passed
@coderabbitai coderabbitai Bot mentioned this pull request Jul 13, 2026
2 of 9 tasks

This branch was successfully deployed

1 active deployment
Preview — ac728587 Deployed Jul 7, 2026 by vercel[bot]
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.

1 participant