rolling-scopes-school / support

15 stars 3 forks source link

Cross-Check GraphQL Basics - UlianaSavi. #906

Closed UlianaSavi closed 1 year ago

UlianaSavi commented 1 year ago
  1. A link to your deployed project
  2. A link to the project repository on Github.
  3. A link to the task
  4. A Link to the checklist for evaluating the task (if it exists)
  5. A screenshot of cross-checking marks изображение
  6. A final score after self-assessment, with comments - 332 / 360 (comments in my PR)
  7. A cross-check score of your Score - 175 / 360
helenakrasnova commented 1 year ago

п. 5. Скриншот с оценками за cross-check (контакты проверяющих должны быть скрыты).

UlianaSavi commented 1 year ago

@helenakrasnova - замазала контакты проверяющих

nosbog commented 1 year ago

Здесь не может быть и речи о повышении балла. Везде, где были дыры в тестах, вы там пролезли. Нет типизации, нет форматирования кода. Много ошибок: Лоудеры должны создаваться под каждый запрос Нет лоудера для подписчиков Неправильно сделанная логика для подписчиков внутри resolver. Во-первых, их может быть >1. Во-вторых, используется неправильная таблица и вы просто переименовываете поле с authorId на id... Поля, которые должны быть nonNull, все опциональные Есть места, где UUIDType не используется, например, в Inputs Нет всех необходимых полей для changeInput-ов

nosbog commented 1 year ago

Я принимаю апелляцию, но только с повышением до 252 (без учета баллов за test-loader).

ThorsAngerVaNeT commented 1 year ago

Выставил оценку