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

Pointing-poker #55

Open
wants to merge 298 commits into
base: main
Choose a base branch
from
Open

Pointing-poker #55

wants to merge 298 commits into from

Conversation

no95typem
Copy link
Owner

@no95typem no95typem commented Oct 3, 2021

MAIN INFO:

🔥 Deploy: https://no95typem.github.io/pp/ 🔥

⭐ Score 450/450. ⭐

🎬 A shot review on youtube 🎬

📧 Please contact us if you have any questions! Email us OR use Discord: no95typem#9889 📧

FULL INFO:

  1. Task: https://github.com/rolling-scopes-school/tasks/blob/master/tasks/react/pointing-poker.md
  2. Screenshot (lobby, dealer):

Screenshot 2021-10-08 at 20-01-18 Pointing Poker

  1. Deploy: https://no95typem.github.io/pp/
  2. Done: 06.10.2021 / deadline 06.10.2021
  3. Score 450/450.

ПОЖАЛУЙСТА, если Ваша оценка не сходится с этой, то прочтите комментарий к расходящемуся пункту. Если после этого вы не изменили решение в нашу пользу, то пожалуйста, оставьте подробный комментарий И СВОИ КОНТАКТЫ.

Ваша оценка - 450 баллов 
Отзыв по пунктам ТЗ:
Выполненные пункты:
1) вёрстка, дизайн, UI страницы приветствия +5 
2) вёрстка, дизайн, UI страницы с настройками +5 
3) вёрстка, дизайн, UI страницы с игрой +5 
4) оригинальный интересный качественный дизайн приложения +10 
5) попап с информацией о пользователе +10 
6) присоединиться/начать игру +5 
7) валидация id игры, имени пользователя +5 
8) страница лобби и вся логика, связанная с лобби(в том часле на странице с настройками) +50 
9) исключить участника голосованием +10 
10) чат +50 
11) валидация настроек +5 
12) сохранение базовых настроек +10 
13) создание списка задач вручную (каждую задачу добавляем отдельно по очереди) +10 
14) генерация списка задач на основе файла +10 
15) список участников +10 
16) список задач +25 
17) таймер +5 
18) среднее значение +5 
19) механизм голосования +50 

Отзыв: Если вы стартовали игру с 0 игроков и без включенной настройки дилер - игрок, то сначала дождитесь первого игрока чтобы начать!

20) карточки переворачиваются автоматически, когда проголосовали все +5 

Отзыв: ПРИ ВКЛЮЧЕННОЙ НАСТРОЙКЕ !

21) повторное голосование +5 
22) добавление задач во время игры +10 

Отзыв: Разрешено только между раундами - то есть только в состоянии ожидания начала раунда!

23) выводится результат в конце игры +5 

Отзыв: Дилер видит результаты всегда, игроки же видят только когда разрешит дилер

24) результат игры можно скачать +40 

Отзыв: Скачивается сразу 2 файла - один для приложения - чтобы можно было открыть его и посмотреть, другой в human readable формате.

25) поддерживает игровую сессию с несколькими пользователями +30 
26) поддерживает несколько игровых сессий одновременно +30 
27) сервер отдаёт корректные ответы, отдаёт HTTP ошибки с нормальными body, по которым можно понять, что произошло, пишет читаемые логи +10 

Отзыв: Сервер работает чисто на ws, в режиме отладки все сообщения лоигруются, так что если желаете проверить - скачивайте репо и запускайте в dev режиме.

28) реализован не указанный в задании дополнительный функционал. Оценивается оригинальная идея, вклад в улучшение качества приложения, полезность, сложность и качество выполнения +20 

Отзыв: 
  1) Темная тема.
  2) Полностью адаптивная верстка работающая с тачем - проверялось на chrome (android).
  3) Звуковые эффекты.
  4) Можно не только экспортировать, но и импортировать сохранение для просмотра результатов в приложении.

29) написано не меньше 10 юнит-тестов, использующих различные методы jest +10 

Vimbi and others added 30 commits September 11, 2021 15:05
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.

3 participants