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

РК #2 #43

Open
3 of 18 tasks
ostapenko-me opened this issue Apr 15, 2019 · 0 comments
Open
3 of 18 tasks

РК #2 #43

ostapenko-me opened this issue Apr 15, 2019 · 0 comments

Comments

@ostapenko-me
Copy link

ostapenko-me commented Apr 15, 2019

Замечания и недоработки

  • Не актуальный README.md
  • Странное описание awekhbebhirubpii
  • Правила игры нужно сверстать, а не использловать картинку
  • Нет пагинации на скорборде (не работает F5)
  • GET https://colors.hackallcode.ru/leaders/4 белый экран
  • Клиентская валидация:

Добавьте на страницы с формами валидацию вводимых данных. Валидация должна работать с помощью JavaScript. То есть если пользователь вводит невалидные данные, необходимо показывать сообщение об ошибках, предупреждения, etc... Сделать это необходимо не с помощью alert'ов, а динамически добавляя на страницу сообщения с текстом ошибки. Примеры того, как это можно оформить, есть в макетах к ДЗ0-2

  • Игра работает неправильно вообще (надо поправить всё)
  • Нет фавиконки
  • Uncaught (in promise) TypeError: Add/AddAll only supports the GET request method.
  • Uncaught (in promise) TypeError: Request failed
  • При ошибках формы вся форма очищается =(
  • Если я авторизован, переход на https://colors.hackallcode.ru/signin не работает (должен сбрасываться url на /)
  • GET https://colors.hackallcode.ru/me в оффлайне отдаёт пустой экран (должен быть редирект на /)
  • Адаптивность 😭
  • Как fs и path попали в package.json?
  • Настройте нормальный линтинг
  • Используйте шаблонные строки :)
  • РРРРРРРРазсепарэйтить компоненты странички!
@hackallcode hackallcode changed the title РК-2 РК #2 May 7, 2019
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

No branches or pull requests

1 participant