sanyamaks / 04-web-application-infrastructure Goto Github PK

Добрый вечер. Благодарю за проделанную работу.
К сожалению, текущая реализация содержит ряд недостатков не позволяющих мне засчитать ее на данной итерации:

Репозиторий

Можно лучше вынести ссылку из заголовка в описание а в заголовке оставить наименование проекта
Нужно исправить: отсутствуют git-теги отражающие номер версии. Создать их можно при помощи команды git tag -a v0.0.1 -m "my first version"
Обращаю внимание что теги по-умолчанию не публикуются при вызове команды git push
Для их публикации выполните команду git push origin --tags
Подробнее: https://git-scm.com/book/ru/v2/%D0%9E%D1%81%D0%BD%D0%BE%D0%B2%D1%8B-Git-%D0%A0%D0%B0%D0%B1%D0%BE%D1%82%D0%B0-%D1%81-%D0%BC%D0%B5%D1%82%D0%BA%D0%B0%D0%BC%D0%B8

Можно лучше: Добавить .gitignore.
.gitignore должен содержать правила, исключающие попадание временных и\или системных файлов в репозиторий.
В случае если вы затрудняетесь с составлением списка базовых правил можете воспользоваться сервисом-генератором, например gitignore.io

Регулярные выражения

Имя

Нужно исправить ограничение наложенное на максимальную длину поля name некорректно. Должно быть - 20 символов, сейчас - 30

Email

Нужно исправить следующие корректные кейсы не проходят валидацию
[email protected] - адрес почты может содержать длинную доменную зону (.academy). Кроме того есть и более длинные зоны :) Рекомендую убрать ограничение на длину сверху или существенно его увеличить
[email protected] - адрес почты может включать в себя произвольное число поддоменов

Нужно исправить следующие некорректные кейсы проходят валидацию
[email protected] - имя ящика не может состоять только из спецсимволов
[email protected] - адрес электронной почты, содержащий несколько символов '-' подряд является некорректным

Можно лучше следующие некорректные кейсы проходят валидацию
[email protected] - имя ящика не может начинаться и заканчиваться на спецсимвол

Номер телефона

Нужно исправить: Данные форматы номеров телефонов проходят валидацию, хотя с точки зрения постановки задачи они являются невалидными.

+7(925 900-90-90
+7925900-90-90
+792590090-90
+7-925-900-90-90
+7 (925) 900-90-90
+7 (925) 9009090
+7(925) 9009090
+7 (925)9009090
+7(925)9009090
+7 925 900-90-90

Адрес сайта

Нужно исправить следующие корректные кейсы не проходят валидацию
http://some.test.yandex.ru - адрес сайта может включать в себя поддомены
https://some.amazing.site.name.academy - адрес сайта может включать в себя длинную доменную зону