CRM ТурФирмы
Веб-программирование
Отзывы заказчиков
Хорошо, приятно имет дело.
5.0
Спасибо за выполненную работу! Александр все сделал согласно договоренностям
5.0
Увы, позитивного сотрудничества не сложилось, и я так и не понял претензий ко мне, но отвечу на них:
1. Было 2 задачи: а) Вывод списка текущих пользователей с ролями, их роли в админ.панели(АП). б) Возможность администратору назначать пользователя менеджером в АП.
Александр сделал страницу со всеми пользователями, на которой можно назначить менеджерскую роль. Но это не подходит для задачи "А" - на странице нельзя посмотреть только тех, кто с менеджерской ролью.
На примере: допустим, нужно посмотреть всех менеджеров. В системе 1000 пользователей, и чтобы собрать список менеджеров - нужно пересмотреть всех 1000 пользователей(50 страниц х 20 пользователей).
Я, просто, вежливо указал на проблему, на что получил негодование и грубость, мол: "в чём сакральный смысл работы, как будут назначаться роли тем у кого нет роли?". Да, текущее решение не решало поставленные задачи, но ведь решение не я же писал, а в ТЗ описаны требования.
Но чтобы избежать переписывания решения - я предложил пользователей с менеджерской ролью просто выводить в самом верху.
В чём моя вина - не понимаю:
1) В ТЗ требования описаны чётко.
2) Я вежливо и без грубостей просто указал, что не решается одна из задач.
3) После предложил альтернативное решение, которое позволило не переписывать существующее.
2. Претензии к коду проекта: "написана ахинея". Но не было конкретики, и проблемы наблюдаются у исполнителя...
1) На просьбу уточнить в чём проблема - ответ не был получен. Утверждение задевало urlManager, но он довольно "стандартный": [https://i.imgur.com/r12W2OH.png]
2) В задачах большая часть взаимодействий идёт с Yii'шным RBAC.
3) Примеры кода проекта были приложены. Я предлагал до начала сотрудничества осмотреть репозиторий.
Но ведь в самом коде Александра есть проблемы. Правда, в обычных случаях, я на это не обращаю внимание, ибо: это всё правится довольно просто, и отличный способ поучиться для программииста/ревьювера.
- DRY: для поиска пользователей с ролью был копирован метод, с добавление строчки orderBy. Статический анализатор IDE это подсветил: [https://i.imgur.com/wLahUe2.png] (но это было исправлено после замечаний)
- В сервисы был затянут Singeltone, хотя там используется контейнер зависимостей, и AuthManager конфигурируется, и мелочи от стат.анализатора IDE отображаются: https://i.imgur.com/d1w8dfs.png
- Из ТЗ: "У пользователя может быть только одна “менеджерская роль”(!) единовременно. " - текущий код Александра отвязывает не только менеджерскую роль, а и любую другую: https://i.imgur.com/1Q1Wbit.png
- Чтобы подгрузить JS для отправки команды роли - используется костыль: https://i.imgur.com/OJ8a783.png
Хотя есть решение из коробки: \yii\web\View::registerJsFile.
- Имена коммитов Git - неинформативны: "admin roles : in progress", "task : in progress", "small changes"
Жаль что так сложилось. Но считаю, что создал максимальные условия для комфортного решения задачи.
1. Было 2 задачи: а) Вывод списка текущих пользователей с ролями, их роли в админ.панели(АП). б) Возможность администратору назначать пользователя менеджером в АП.
Александр сделал страницу со всеми пользователями, на которой можно назначить менеджерскую роль. Но это не подходит для задачи "А" - на странице нельзя посмотреть только тех, кто с менеджерской ролью.
На примере: допустим, нужно посмотреть всех менеджеров. В системе 1000 пользователей, и чтобы собрать список менеджеров - нужно пересмотреть всех 1000 пользователей(50 страниц х 20 пользователей).
Я, просто, вежливо указал на проблему, на что получил негодование и грубость, мол: "в чём сакральный смысл работы, как будут назначаться роли тем у кого нет роли?". Да, текущее решение не решало поставленные задачи, но ведь решение не я же писал, а в ТЗ описаны требования.
Но чтобы избежать переписывания решения - я предложил пользователей с менеджерской ролью просто выводить в самом верху.
В чём моя вина - не понимаю:
1) В ТЗ требования описаны чётко.
2) Я вежливо и без грубостей просто указал, что не решается одна из задач.
3) После предложил альтернативное решение, которое позволило не переписывать существующее.
2. Претензии к коду проекта: "написана ахинея". Но не было конкретики, и проблемы наблюдаются у исполнителя...
1) На просьбу уточнить в чём проблема - ответ не был получен. Утверждение задевало urlManager, но он довольно "стандартный": [https://i.imgur.com/r12W2OH.png]
2) В задачах большая часть взаимодействий идёт с Yii'шным RBAC.
3) Примеры кода проекта были приложены. Я предлагал до начала сотрудничества осмотреть репозиторий.
Но ведь в самом коде Александра есть проблемы. Правда, в обычных случаях, я на это не обращаю внимание, ибо: это всё правится довольно просто, и отличный способ поучиться для программииста/ревьювера.
- DRY: для поиска пользователей с ролью был копирован метод, с добавление строчки orderBy. Статический анализатор IDE это подсветил: [https://i.imgur.com/wLahUe2.png] (но это было исправлено после замечаний)
- В сервисы был затянут Singeltone, хотя там используется контейнер зависимостей, и AuthManager конфигурируется, и мелочи от стат.анализатора IDE отображаются: https://i.imgur.com/d1w8dfs.png
- Из ТЗ: "У пользователя может быть только одна “менеджерская роль”(!) единовременно. " - текущий код Александра отвязывает не только менеджерскую роль, а и любую другую: https://i.imgur.com/1Q1Wbit.png
- Чтобы подгрузить JS для отправки команды роли - используется костыль: https://i.imgur.com/OJ8a783.png
Хотя есть решение из коробки: \yii\web\View::registerJsFile.
- Имена коммитов Git - неинформативны: "admin roles : in progress", "task : in progress", "small changes"
Жаль что так сложилось. Но считаю, что создал максимальные условия для комфортного решения задачи.
2.3
Yii2 натянуть страницу статистики
1500 UAH
Основная задача была сделана, осталось пару правок сделать, но исполнитель требует закрыть заказ, мне придется самому внести правки. Видно, что программист отлично знает свое дело. Рекомендую
4.3
Задание выполнено, исполнитель был на связи.
4.0
Изменить дорогие модули
Персональный
2500 UAH
Все хорошо и профессионально выполнено с учетом всех предложений
5.0
Доработать сайт на YII1.1 по ТЗ
3396 UAH
Не первый раз работаю с этим специалистом, как всегда, всё на уровне! Зачёт!
5.0
Выгрузка товар на сервис tiu.ru
600 UAH
Все отлично, работа выполненна качественно, рекомендую!
5.0
Поправить фид данних
200 UAH
Спасибо за проделанную работу
5.0
Yii2 Доработки системы поиска туров
3400 UAH
Спасибо за быструю и хорошо выполненную работу!
Я возвращаюсь уже второй раз и в дальнейшем при необходимости обращаюсь!
Я возвращаюсь уже второй раз и в дальнейшем при необходимости обращаюсь!
5.0
Дописати сайт (написаний на PHP Framework Yii2)
Персональный
1200 UAH
Дякую Олександру за гарно виконану роботу! Рекомендую!
5.0
Выгрузка с 1с на ocStore
800 UAH
Все хорошо. проект выполнен, Выгрузка с 1с на интернет магазин работает.
5.0
Было сложное задание, довёл до конца.
Спасибо!
Спасибо!
5.0
Настроить кореектную работу почты на yii2
Персональный
400 UAH
Возникли сложности с реализацией задачи, но Александр предложил альтернативу и успешно реализовал
5.0
Исполнитель быстро решил проблему, предложил несколько вариантов
5.0
Доработка сайта php/yii
Персональный
1274 UAH
Отличная работа, очень доволен! Квалифицировано, быстро, профессионально!
5.0
Форма на Yii2 (доработка)
Персональный
1100 UAH
Спасибо за качественно выполненную работу, будем сотрудничать еще
5.0
Wordpress
30 000 UAH
Работаю не первый раз, все отлично, всем советую, парень просто золотой, все консультирует, берет за свою работу шекелями, все прекрасно!
5.0
Доработать сайт на yii2 до рабочего состояния
Персональный
1070 UAH
Доработали проект, все хорошо в итоге.
5.0
YII налаштування 2
Персональный
600 UAH
Вдячна за швидко виконану роботу!
Роботою задоволена!
Роботою задоволена!