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 · 5 comments
Open

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

physphile opened this issue Jul 5, 2023 · 5 comments
Assignees
@BatuevIO
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
@physphile physphile moved this to In Progress in Viribus Team Jul 5, 2023
@grigoriev-semyon grigoriev-semyon moved this from In Progress to Todo in Viribus Team Jul 5, 2023
@grigoriev-semyon grigoriev-semyon moved this from Todo to In Progress in Viribus Team Jul 5, 2023
@grigoriev-semyon
Copy link
Member

related to profcomff/auth-api#128

@Arzangulyan Arzangulyan moved this from In Progress to Todo in Viribus Team Jul 20, 2023
@Arzangulyan Arzangulyan moved this from Todo to In Progress in Viribus Team Jul 20, 2023
@Arzangulyan
Copy link
Member

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

@Arzangulyan Arzangulyan moved this from In Progress to Todo in Viribus Team Aug 10, 2023
@Arzangulyan
Copy link
Member

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

@Arzangulyan Arzangulyan moved this from Todo to Backlog in Viribus Team Mar 11, 2024
@dyakovri
Copy link
Member

dyakovri commented Apr 9, 2024

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

@BatuevIO
Copy link
Contributor

Нужно пересмотреть ишью с учетом взаимодействия с апи через api-uilib -- там ошибки нужно ловить через структуру if (data) {} else if (error) {}, либо через response. В большинстве мест это написано, но нужно еще раз внимательно посмотреть и потестить.

@BatuevIO BatuevIO assigned BatuevIO and unassigned physphile and Temmmmmo Nov 11, 2024
@BatuevIO BatuevIO mentioned this issue Nov 11, 2024
Closed
16 tasks
@BatuevIO BatuevIO mentioned this issue Nov 20, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@BatuevIO BatuevIO

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

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