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

[FEATURE REQUEST] Автоматическое обновление расписания #59

Closed
ValeryVerkhoturov opened this issue Aug 28, 2021 · 2 comments · Fixed by #74 or #78
Closed

[FEATURE REQUEST] Автоматическое обновление расписания #59

ValeryVerkhoturov opened this issue Aug 28, 2021 · 2 comments · Fixed by #74 or #78
Labels
enhancement New feature or request

Comments

@ValeryVerkhoturov
Copy link

ValeryVerkhoturov commented Aug 28, 2021 •
edited
Loading

Проблемы

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

Решение

  • Обновлять расписание при входе в приложение. В случае неудачи подгрузить из памяти устройства, но пытаться загрузить и заменить старое на расписание из Интернета.
  • Убрать значок обновления расписания

Альтернативы
Изменять цвет значка обновления

  • удачная загрузка - белый, неудачная - красный
  • удачная загрузка - серый, неудачная - белый

Скрывать значок обновления в случае удачной подгрузки с Интернета
@ValeryVerkhoturov ValeryVerkhoturov added the enhancement New feature or request label Aug 28, 2021
@uSlashVlad uSlashVlad mentioned this issue Sep 3, 2021
Merged
@L3odr0id L3odr0id linked a pull request Sep 4, 2021 that will close this issue
FEATURE: succesful server loading indication #74
Merged
@0niel
Copy link
Owner

0niel commented Sep 4, 2021

Нет представления, подгружено расписание с сервера или из памяти устройства.

Проблема исправлена в #74 от @uSlashVlad, но считаю, что решение нужно дополнительно какое-то сообщение пользователю, так как цвет может быть не очень понятен.

@uSlashVlad uSlashVlad mentioned this issue Sep 5, 2021
Merged
@uSlashVlad
Copy link
Contributor

Полагаю, что #71, #74 и #78 полностью закрывают все пункты данной фичи и поэтому думаю, что и задачу можно закрыть

@0niel 0niel linked a pull request Sep 6, 2021 that will close this issue
FEATURE: schedule from cache indication #78
Merged
@0niel 0niel closed this as completed Sep 6, 2021
@0niel 0niel mentioned this issue Sep 8, 2021
Merged
@0niel 0niel added this to App Feb 20, 2024
@0niel 0niel moved this to Done in App Feb 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
App
Status: Done
Milestone
No milestone
3 participants
@uSlashVlad @0niel @ValeryVerkhoturov