Borys Holub

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

#️⃣ Вступление

Задача была “тестовой”, чтобы просто познакомиться с исполнителем перед началом объёмного сотрудничества.

Перед утверждением условий:
- Предоставил Борису полное ТЗ
- Доступ в Репозиторий проекта

Разработчик сказал, что код в целом хороший и чистый, рассказал о требованиях к больших проектам(все задачи только ему, чтобы другие не писали “г*-код”; если код грязный — ставка больше).

Борис самостоятельно назначил стоимость задачи и срок, как он написал, с запасом. Утвердили.

Проблемы:

1️⃣ Ближе к концу проекта Борис стал требовать доплату, поскольку “потратил больше времени, чем планировал”.

Обозначу, в вопросах доплаты всегда готов идти на встречу:
- часто вношу бонус, если исполнитель действительно добавил новых улучшений(пример: соседний проект с Александром)
- задача оказалась сложнее и исполнитель её недооценил
- все дополнительные задачи идут отдельными проектами за отдельную плату, что можно увидеть по отзывам

Но в данном случае — с выставленными требованиями не согласен по следующим причинам:

1. Борис буквально отписывал почти каждый свой шаг в проекте, а позже рассказывал о себе, его стиле работы. Возможно, это интересно, но рамках мелкой задачи — это излишне, и только тратит наше обоюдное время: наш диалог с 1-ой малой задачей, больше чем диалог с постоянными исполнителями, где 4-5 средних-больших.
Полагаю, что на “монолог” было потрачено больше времени, чем на разработку.

2. Стал конфигурировать и копаться в тех частях приложения, с которыми не связана задача, на что я вежливо указывал.

3. Борис сообщал, что были “Требования вне ТЗ и он ещё реализовал улучшения”.
При уточнение было названо: “Не было инструкции по 2FA, пришлось её изучать и обходить; Не было аккаунта ЮниСендер для теста; Пофикшен баг;”

Но указанное - никак не связано с дополнительными требованиями. ТЗ не изменял. Также:
1) "Улучшения". В ТЗ было указано: необходимо выводить отправленные письма администраторами, чтобы контролировать их содержимое.
В системе уже был лог отправленных писем через Почтовый сервис. Борис, с моим одобрением, просто туда стал записывать письма отправленные администратором, и выводить их содержимое.
Фактически это не улучшение, а упрощение работы самому Борису - ему не пришлось делать Read-функционал.
2) Инструкции были в README, комментариях к параметрам.
☑️ Чтобы упростить работу разработчикам — сделаем ещё несколько отдельных разделов в README для конфигурации.
3) Следовало было запросить тестовые данные у меня. а) В условиях проектах специально прописано — не стесняйтесь спрашивать; б) Обычно я самостоятельно такие тестовые данные кидаю, но на этот раз упустил.
☑️ Добавил в чек-лист перед стартом проекта — выдавать демо-данные, чтобы больше не упускать.
4) Благодарен за фиксацию бага. Но указанный баг тесно связан с серверным окружением Бориса, и даже не в той части приложения, с которой следовало вести работу. Обнаружен он был во временном решении. Фиксация - обычная проверка на существование, причём правкой была затёрта TODO: который помечал временное решение;
☑️ Добавим Docker в проект, чтобы все разрабатывали с одним ENV.

4. Все детали проекта были видны до сотрудничества. Борис сам установил оценку, срок и ещё заложил запас, как сам написал. Я даже уточнял в поддержке за правоту в этой ситуации.

2️⃣ Отношение к работе/качество кода

Борис часто говорит о качестве кода(см. раздел "Вступление", абзац №3). Но о своих ошибках пишет - "Это неважно, это мелочь". Считаю, что такой подход несправедливый.
А ошибки в любом случае нестрашны, и повод поучиться для всех, тем более мы подряжаем тестера для перестраховки.

Но раз на то пошло:
1. Был отключен HTML-Encode в GridView, выводятся RAW-данные без фильтрации(зачем?). Username вводится пользователем, поэтому может содержать код для XSS-атаки(+длинная строки): https://i.imgur.com/ueToBOV.png
2. Чтобы выводить данных о пользователях, используются костыли, вместо связей(relations) БД: https://i.imgur.com/cg1HyEM.png || https://i.imgur.com/wqbMM6P.png
3. Не описан revert миграций, хотя он возможен с помощью dropColumn: https://i.imgur.com/Fx7ywLK.png
4. Жёсткая привязка к ID отправителя к авторизованному пользователю в Сервисе, что делает его непереиспользуемым, если ID нужно указать самостоятельно: https://i.imgur.com/XYrZlU4.png
5. Не используется TypeHinting в GridView, что делает неудобным переименование свойств через "Refactor": https://i.imgur.com/0fj30su.png
6. Раздувание сущности Mail. Фактически ему было привязано 3 опицональных поля, которые есть только в сценарии пользователя администратором: https://i.imgur.com/azhhfpD.png
Вместо того, чтобы сделать отдельную сущность и присвоить ей связь. А если потребуются ещё дополнительное поля для ручной отправки?

3️⃣ Остальное
На простые уточняющие вопросы, касательно "улучшений и задач вне ТЗ", чтобы понять позицию - получил претензии.
1. "Долгая проверка, долгие ответы" - у меня есть свой перечень задач, я не могу реагировать 24/7 и ни от кого этого не требую. А также время потребовалось на уточнение вопросов касательно "задач вне ТЗ" и чтение всего потока сообщений.
2. "Понятно почему над проектом работают столько людей и не задерживаются" - в проекте 2 контирбютора, помимо меня. Один занят другой задачей, второй в зоне Б/Д и испытывает проблемы со связью.

thank you. Professionally and quickly the artist wrote a script on the collection of data. I recommend it.

Отлично выполненная работа! Рекомендую, скорость на высоте.

It was pleasant to work. A little delayed the project on our fault, in general everything is fine.)

SITEMAP map, all clearly by TZ

Thanks for the great work) There was much more work done than planned, for which a special thanks to the freelancer =)

Работа была выполнена своевременно и качественно.

Work can be said to have done, but the approach to work leaves you want better.There was a moment when I found an error, on my 13 inch mac, the distance between the photo was incorrectly visually displayed, told me that the problem in the operating system and that it was a separate work for the dop bone, after a large number of letters wrote that I still found what the problem and corrected.In the professionalism he put 3, more than the way of communication.After giving up the work I wrote that I will add a photo (they there are just about 80) and check, not passed and 24 hours, how I get from the executor a threat that he will address to the arbitration, direct speech - "You will check it?I’m waiting for the second day to check.Let it be honest: if within 2 hours you don’t finish the check and don’t scratch, then I’m calling to the arbitration.So that there is no harassment later.I warned in advance.”

When I wrote that I have questions how the mobile version is displayed, he replied that the upgrades are already for the dop bone, I clarified how much, after knowing the cost I refused, after that the executor goes to the arbitration, accusing me that I do not agree to pay for the dop work and that I am constantly changing the requirements, bringing a direct speech -
“The customer is constantly changing requirements.Even when everything is done and matches the TZ, he wants more free donations and does not agree to the add-on.I have exhausted the original TZ several times and spent time more times than the result will be paid.”