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.

Sign up for GitHub

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

Jump to bottom

Обработка ошибок апи #116

Open
physphile opened this issue Jul 5, 2023 · 4 comments
Open

Обработка ошибок апи #116

physphile opened this issue Jul 5, 2023 · 4 comments
Assignees
@physphile @Temmmmmo
Labels
good first issue 👶 Задача подходит для новичков new feature 🆕 Новая фича или запрос на нее

Comments

@physphile
Copy link
Contributor

physphile commented Jul 5, 2023 •
edited
Loading

Все вызовы апи должны быть обернуты в try/catch. Ошибки должны быть типизированы. Все обращения к синглтонам authApi, marketingApi , timetableApi и т. п. должны выполняться только внутри соответствующих контроллеров, которые находятся в директории /api/controllers.
@physphile physphile self-assigned this Jul 5, 2023
@grigoriev-semyon grigoriev-semyon added good first issue 👶 Задача подходит для новичков new feature 🆕 Новая фича или запрос на нее labels Jul 5, 2023
@grigoriev-semyon
Copy link
Member

related to profcomff/auth-api#128

@Arzangulyan
Copy link
Member

@physphile, а в каком статусе задача осталась перед твоим отъездом?

@Arzangulyan
Copy link
Member

#142
Пока еще не все доделано

@dyakovri
Copy link
Member

dyakovri commented Apr 9, 2024

В том виде, что сделано сейчас, создает много проблем. Иногда ты можешь получить ошибку и об этом не нужно уведомлять пользователя, а показать что-то на странице.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@physphile physphile

@Temmmmmo Temmmmmo

Labels
good first issue 👶 Задача подходит для новичков new feature 🆕 Новая фича или запрос на нее
Projects
Viribus Team
Status: Backlog
Milestone
No milestone
Development

No branches or pull requests
5 participants
@dyakovri @Arzangulyan @grigoriev-semyon @physphile @Temmmmmo