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

Gw 86 #63

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Gw 86 #63

wants to merge 11 commits into from

Conversation

console1928
Copy link
Collaborator

@console1928 console1928 commented Jun 28, 2021

Задача GW-86 в trello.com

Что было сделано в задаче

  1. Поправлено форматирование текста в тренажере
  2. Добавлена логика удаления cookie gw_email, если сессия пользователя стала невалидной на бэке
  3. Добавлен список реальных профессий
  4. Поправлен автоматический скролл до нижней части страницы теста
  5. Удален лишний запрос к апи на странице теста
  6. Поправлено отображение правильно/неправильно отмеченных чекбоксов/радиокнопок
  7. Поправлена клавиатурная навигация на страницах регистрации и входа (tab теперь активирует переход только между полями ввода)
  8. Добавлено скрытие хедера при скролле страницы вниз
  9. Поправлен баг, из-за которого после выхода и входа в тренажер заново у пользователя отображалось меню предыдущего пользователя
  10. Добавлен актуальный favicon

Как протестировать

Перейти в браузере на страницу приложения, попробовать воспроизвести описанные баги

Скриншоты, если были изменения в верстке

bugfixes

Чеклист для проверки

  • Ветка называется GW-[номер задачи из трелло]. Для задачи https://trello.com/c/cjbG0nCi/44-test ветка будет называться GW-44
  • Задача в трелло стоит в колонке review
  • Этот PR открыт только для одной задачи
  • Все коммиты пишутся как GW-[номер задачи из трелло] суть изменений, например: GW-44 add test task
  • Нет лишних коммитов, типа fix, tmp
  • Базовая ветка установлена правильно, в PR нет чужих коммитов и мержей

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant