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

Добавить обертку над апи #54

Open
3 tasks
Shiplayer opened this issue Dec 25, 2020 · 2 comments
Open
3 tasks

Добавить обертку над апи #54

Shiplayer opened this issue Dec 25, 2020 · 2 comments
Labels
feature
Milestone
MainActivity - 2

Comments

@Shiplayer
Copy link
Contributor

  • Обертка должна уметь логировать и обрабатывать сетевые ошибки
  • Обновлять токен, если он протух и повторять запрос
  • Повторять запросы раз в n секунд, если нет интернета
@temnik15
Copy link
Collaborator

Вопрос по пункту 1: зачем нам здесь логгировать сетевые ошибки если они уже логируются внутри Interceptor?
Вопрос по пункту 3: раз в n секунд до тех пор что? пока не появится сеть? или же все таки спустя m секунд прекращает попытки и показываем уведомление для пользователя?

@Shiplayer
Copy link
Contributor Author

Shiplayer commented Dec 25, 2020 •
edited
Loading

ответ 1. обработав ошибки, мы уже можем не пробрасывать их ниже по цепочке и возвращать что-то определенное
ответ 3. давай посмотрим по обстоятельствам, если будет не сложно сделать, чтобы после 5 попыток отвалился, то будет классно

@temnik15 temnik15 assigned temnik15 and unassigned temnik15 Dec 26, 2020
@Shiplayer Shiplayer added this to the MainActivity - 2 milestone Feb 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature
Projects
None yet
Milestone
MainActivity - 2
Development

No branches or pull requests
2 participants
@Shiplayer @temnik15