Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Рефакторинг эндпоинтов #6

Open
AleksanderNekr opened this issue Oct 24, 2024 · 0 comments
Open

Рефакторинг эндпоинтов #6

AleksanderNekr opened this issue Oct 24, 2024 · 0 comments
Assignees
Labels
wontfix This will not be worked on

Comments

@AleksanderNekr
Copy link
Collaborator

AleksanderNekr commented Oct 24, 2024

Перенести эндпоинты в отдельные файлы

Проблема: текущие эндпоинты находятся в файле Program.cs и занимают много места, также их нужно отрефакторить

Сделать:

  1. Перенести эндпоинты авторизации в папку Auth файл Endpoints.cs по примеру – документационные комменты пока лучше не добавлять, чтоб не тратить время
  2. Добавить эндпоинт по обновлению токенов: см. предыдущую ссылку эндпоинт /tokens
  3. В ответах на вход и регистрацию не хватает инфы о пользователе
  4. Вынести ответы на вход и регистрацию в отдельный класс в папку Auth
  5. В модель ответа инфы о пользователе добавить его id
  6. Файлы LoginRequest и RegisterRequest также унести в папку Auth
  7. Все эндпоинты должны начинаться с пути /api (задан как базовый для всех эндпоинтов через api = app.MapGroup("api"); в Program.cs
  8. Эндпоинт /user должен перенестись в папку Users в файл Endpoints.cs и тоже начинается с /api, туда же стоит перенести всё из Domain, а Domain убрать
  9. Эндпоинт /ping остается там же, но должен начинаться с /api тоже

Переходим к Vertical Slice архитектуре

@AleksanderNekr AleksanderNekr added the cifra-feature Хакатон "Цифра" label Oct 24, 2024
@AleksanderNekr AleksanderNekr changed the title Рефакторинг эндпоинтов авторизации Рефакторинг эндпоинтов Oct 24, 2024
@AleksanderNekr AleksanderNekr added wontfix This will not be worked on and removed cifra-feature Хакатон "Цифра" labels Oct 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
Status: Todo
Development

No branches or pull requests

2 participants