rolling-scopes-school / support

15 stars 3 forks source link

Cross-Check 'momentum' - 'Olgasinkevich'. #311

Closed Olgasinkevich closed 3 years ago

Olgasinkevich commented 3 years ago

Deploy: https://olgasinkevich.github.io/momentum/index.html Ссылка на репозиторий с проектом на GitHub: https://github.com/Olgasinkevich/momentum Task: https://github.com/rolling-scopes-school/tasks/blob/master/tasks/momentum/momentum-stage1.md Ссылка на форму для проверки проекта: такой нет. Скриншот с оценками за cross-check (контакты проверяющих должны быть скрыты). Submit 4 Submit 3 Submit 2 Submit 1

Оценка в результате самопроверки с ее аргументацией: 85/150: сделаны часы и календарь (+15), приветствие (+10), фоновое изображение со ссылкой на API(+20, +10), погода (+15), цитата (+15). Возникла проблема с тем, что я, к сожалению, забыла форкнуть репозиторий с фоновыми изображениями на свой github. В школьном репозитории была переименована папка 'day' и после этого фоновые изображения перестали открываться с 12 дня до 18 часов. В консоли видно, что идет запрос на ресурс и выдается ошибка. Также в другое время суток фоновое изображение отображается. Проверяющие студенты согласны, что js по фоновым изображениям работает и вопрос в запросе к уже не существующей директории (backend), однако после дедлайна оценку можно исправить только путем апелляции. Очень надеюсь на ваше понимание и снисходительность, потому что в моем случае эти дополнительные баллы почти +50% к score за задание и это очень важно для меня. Спасибо. Балл за cross-check из вашего Score: 55/150.

AlekseyVY commented 3 years ago

Ваша оценка - 60 баллов

Отзыв по пунктам ТЗ: Не выполненные/не засчитанные пункты: 1) время выводится в 24-часовом формате, например: 21:01:00

2) выводится уведомление об ошибке при вводе некорректных значений, для которых API не возвращает погоду (пустая строка или бессмысленный набор символов)

3) при клике по кнопке Play/Pause проигрывается первый трек из блока play-list, иконка кнопки меняется на Pause

4) при клике по кнопке Play/Pause во время проигрывания трека, останавливается проигрывание трека, иконка кнопки меняется на Play

5) треки пролистываются по кругу - после последнего идёт первый (клик по кнопке Play-next), перед первым - последний (клик по кнопке Play-prev)

6) трек, который в данный момент проигрывается, в блоке Play-list выделяется стилем

7) после окончания проигрывания первого трека, автоматически запускается проигрывание следующего. Треки проигрываются по кругу: после последнего снова проигрывается первый.

8) добавлен прогресс-бар в котором отображается прогресс проигрывания

9) при перемещении ползунка прогресс-бара меняется текущее время воспроизведения трека

10) над прогресс-баром отображается название трека

11) отображается текущее и общее время воспроизведения трека

12) есть кнопка звука при клике по которой можно включить/отключить звук

13) добавлен регулятор громкости, при перемещении ползунка регулятора громкости меняется громкость проигрывания звука

14) можно запустить и остановить проигрывания трека кликом по кнопке Play/Pause рядом с ним в плейлисте

15) переводится язык и меняется формат отображения даты

16) переводится приветствие и placeholder

17) переводится прогноз погоды в т.ч описание погоды и город по умолчанию

18) переводится цитата дня т.е цитата отображается на текущем языке приложения. Сама цитата может быть другая

19) переводятся настройки приложения, при переключении языка приложения в настройках, язык настроек тоже меняется

20) в качестве источника изображений может использоваться Unsplash API

21) в качестве источника изображений может использоваться Flickr API

22) в настройках приложения можно указать язык приложения (en/ru или en/be)

23) в настройках приложения можно указать источник получения фото для фонового изображения: коллекция изображений GitHub, Unsplash API, Flickr API

24) если источником получения фото указан API, в настройках приложения можно указать тег/теги, для которых API будет присылает фото

25) в настройках приложения можно скрыть/отобразить любой из блоков, которые находятся на странице: время, дата, приветствие, цитата дня, прогноз погоды, аудиоплеер, список дел/список ссылок/ваш собственный дополнительный функционал

26) Скрытие и отображение блоков происходит плавно, не влияя на другие элементы, которые находятся на странице, или плавно смещая их

27) настройки приложения сохраняются при перезагрузке страницы

28) ToDo List - список дел (как в оригинальном приложении) или Список ссылок (как в оригинальном приложении) или Свой собственный дополнительный функционал, по сложности аналогичный предложенным

Выполненные пункты: 1) время обновляется каждую секунду - часы идут. Когда меняется одна из цифр, остальные при этом не меняют своё положение на странице (время не дёргается)

2) выводится день недели, число, месяц, например: "Воскресенье, 16 мая" / "Sunday, May 16" / "Нядзеля, 16 траўня"

3) текст приветствия меняется в зависимости от времени суток (утро, день, вечер, ночь). Проверяется соответствие приветствия текущему времени суток

4) пользователь может ввести своё имя. При перезагрузке страницы приложения имя пользователя сохраняется

5) ссылка на фоновое изображение формируется с учётом времени суток и случайного номера изображения (от 01 до 20). Проверяем, что при перезагрузке страницы фоновое изображение изменилось. Если не изменилось, перезагружаем страницу ещё раз

6) изображения можно перелистывать кликами по стрелкам, расположенным по бокам экрана.Изображения перелистываются последовательно - после 18 изображения идёт 19 (клик по правой стрелке), перед 18 изображением идёт 17 (клик по левой стрелке)

7) изображения перелистываются по кругу: после двадцатого изображения идёт первое (клик по правой стрелке), перед 1 изображением идёт 20 (клик по левой стрелке)

8) при смене слайдов важно обеспечить плавную смену фоновых изображений. Не должно быть состояний, когда пользователь видит частично загрузившееся изображение или страницу без фонового изображения. Плавную смену фоновых изображений не проверяем: 1) при загрузке и перезагрузке страницы 2) при открытой консоли браузера 3) при слишком частых кликах по стрелкам для смены изображения

9) при перезагрузке страницы приложения указанный пользователем город сохраняется, данные о нём хранятся в local storage

10) для указанного пользователем населённого пункта выводятся данные о погоде, если их возвращает API. Данные о погоде включают в себя: иконку погоды, описание погоды, температуру в °C, скорость ветра в м/с, относительную влажность воздуха в %. Числовые параметры погоды округляются до целых чисел

11) при загрузке страницы приложения отображается рандомная цитата и её автор

12) при перезагрузке страницы цитата обновляется (заменяется на другую). Есть кнопка, при клике по которой цитата обновляется (заменяется на другую)

Olgasinkevich commented 3 years ago

Здравствуйте, @AlekseyVY. Спасибо большое за уделённое время! Подскажите, пожалуйста, почему не защитано,что часы выводятся в 24:00:00 формате?😳 screen time По моим подсчётам по ведомости проверки за часы, календарь, приветствие, цитату, погоду получается 85 баллов. Вечером ещё один студент пересмотрел работы и поставил 80, т к в начале не заметил наличие фонового изображения... Я успеваю не все по курсу, но здесь я старалась, поэтому так переживаю за баллы)...

пт, 29 окт. 2021 г., 20:54 Aleksey @.***>:

Ваша оценка - 60 баллов

Отзыв по пунктам ТЗ: Не выполненные/не засчитанные пункты:

1.

время выводится в 24-часовом формате, например: 21:01:00 2.

выводится уведомление об ошибке при вводе некорректных значений, для которых API не возвращает погоду (пустая строка или бессмысленный набор символов) 3.

при клике по кнопке Play/Pause проигрывается первый трек из блока play-list, иконка кнопки меняется на Pause 4.

при клике по кнопке Play/Pause во время проигрывания трека, останавливается проигрывание трека, иконка кнопки меняется на Play 5.

треки пролистываются по кругу - после последнего идёт первый (клик по кнопке Play-next), перед первым - последний (клик по кнопке Play-prev) 6.

трек, который в данный момент проигрывается, в блоке Play-list выделяется стилем 7.

после окончания проигрывания первого трека, автоматически запускается проигрывание следующего. Треки проигрываются по кругу: после последнего снова проигрывается первый. 8.

добавлен прогресс-бар в котором отображается прогресс проигрывания 9.

при перемещении ползунка прогресс-бара меняется текущее время воспроизведения трека 10.

над прогресс-баром отображается название трека 11.

отображается текущее и общее время воспроизведения трека 12.

есть кнопка звука при клике по которой можно включить/отключить звук 13.

добавлен регулятор громкости, при перемещении ползунка регулятора громкости меняется громкость проигрывания звука 14.

можно запустить и остановить проигрывания трека кликом по кнопке Play/Pause рядом с ним в плейлисте 15.

переводится язык и меняется формат отображения даты 16.

переводится приветствие и placeholder 17.

переводится прогноз погоды в т.ч описание погоды и город по умолчанию 18.

переводится цитата дня т.е цитата отображается на текущем языке приложения. Сама цитата может быть другая 19.

переводятся настройки приложения, при переключении языка приложения в настройках, язык настроек тоже меняется 20.

в качестве источника изображений может использоваться Unsplash API 21.

в качестве источника изображений может использоваться Flickr API 22.

в настройках приложения можно указать язык приложения (en/ru или en/be) 23.

в настройках приложения можно указать источник получения фото для фонового изображения: коллекция изображений GitHub, Unsplash API, Flickr API 24.

если источником получения фото указан API, в настройках приложения можно указать тег/теги, для которых API будет присылает фото 25.

в настройках приложения можно скрыть/отобразить любой из блоков, которые находятся на странице: время, дата, приветствие, цитата дня, прогноз погоды, аудиоплеер, список дел/список ссылок/ваш собственный дополнительный функционал 26.

Скрытие и отображение блоков происходит плавно, не влияя на другие элементы, которые находятся на странице, или плавно смещая их 27.

настройки приложения сохраняются при перезагрузке страницы 28.

ToDo List - список дел (как в оригинальном приложении) или Список ссылок (как в оригинальном приложении) или Свой собственный дополнительный функционал, по сложности аналогичный предложенным

Выполненные пункты:

1.

время обновляется каждую секунду - часы идут. Когда меняется одна из цифр, остальные при этом не меняют своё положение на странице (время не дёргается) 2.

выводится день недели, число, месяц, например: "Воскресенье, 16 мая" / "Sunday, May 16" / "Нядзеля, 16 траўня" 3.

текст приветствия меняется в зависимости от времени суток (утро, день, вечер, ночь). Проверяется соответствие приветствия текущему времени суток 4.

пользователь может ввести своё имя. При перезагрузке страницы приложения имя пользователя сохраняется 5.

ссылка на фоновое изображение формируется с учётом времени суток и случайного номера изображения (от 01 до 20). Проверяем, что при перезагрузке страницы фоновое изображение изменилось. Если не изменилось, перезагружаем страницу ещё раз 6.

изображения можно перелистывать кликами по стрелкам, расположенным по бокам экрана.Изображения перелистываются последовательно - после 18 изображения идёт 19 (клик по правой стрелке), перед 18 изображением идёт 17 (клик по левой стрелке) 7.

изображения перелистываются по кругу: после двадцатого изображения идёт первое (клик по правой стрелке), перед 1 изображением идёт 20 (клик по левой стрелке) 8.

при смене слайдов важно обеспечить плавную смену фоновых изображений. Не должно быть состояний, когда пользователь видит частично загрузившееся изображение или страницу без фонового изображения. Плавную смену фоновых изображений не проверяем: 1) при загрузке и перезагрузке страницы 2) при открытой консоли браузера 3) при слишком частых кликах по стрелкам для смены изображения 9.

при перезагрузке страницы приложения указанный пользователем город сохраняется, данные о нём хранятся в local storage 10.

для указанного пользователем населённого пункта выводятся данные о погоде, если их возвращает API. Данные о погоде включают в себя: иконку погоды, описание погоды, температуру в °C, скорость ветра в м/с, относительную влажность воздуха в %. Числовые параметры погоды округляются до целых чисел 11.

при загрузке страницы приложения отображается рандомная цитата и её автор 12.

при перезагрузке страницы цитата обновляется (заменяется на другую). Есть кнопка, при клике по которой цитата обновляется (заменяется на другую)

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/rolling-scopes-school/support/issues/311#issuecomment-954936209, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQ4O42UQFMIEYVHU2CTLJEDUJLNV3ANCNFSM5G7MFHXA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

AlekseyVY commented 3 years ago

1

Olgasinkevich commented 3 years ago

@AlekseyVY, простите, но у меня время отображается в 24-часовом формате. Я прикрепила скрин всего экрана выше в комментарии. Как такое может быть?

AlekseyVY commented 3 years ago

Это вашь код, не мой.

Olgasinkevich commented 3 years ago

@AlekseyVY к сожалению, 1) я не вижу, что это конкретно мое приложение, так как скрин показывает только часть 2)я еще не разработчик на проекте, который получает зарплату за поиск и устранение багов, а студент на курсе, поэтому была бы очень благодарна за обратную связь и хотя бы предположение, почему именно у Вас время отображается не так, ведь до этого минимум у меня и 4 человек все было хорошо? Другой браузер? Настройки? Возможно, время все же отображается в формате 24:00:00 и почему-то у Вас появляется приставка "AM"? Та проблема, с которой я открывала апелляцию, давала +30 баллов. Вы же добавили только 5 баллов, констатируя, что проблемы с изображениями нет. -25 баллов за неверное отображение времени, проблемы с которым до сих пор ни у кого не возникало, при том, что максимум за время +15 баллов?