AM
Попробовал через мобильное приложение ревью-коммент вставить. Теперь не понимаю, на какую строчек он уставился и как так получилось. Вроде пальцем целился в другую 😁
Вот нефиг ревью с мобильника проводить. scope не тот :)
Size: a a a
2020 July 15
А
Вот нефиг ревью с мобильника проводить. scope не тот :)
Ну мелкомягкие приложение начали развивать. Грех не попробовать. Тем более этот код на компе уже смотрел
AM
Ну мелкомягкие приложение начали развивать. Грех не попробовать. Тем более этот код на компе уже смотрел
Я из Chrome иногда комменчу с мобильника.
AM
Но да, попасть надо ещё.
AM
Так... я всё-таки за yii-event
AM
yii- потому что вне Yii оно не применимо
AM
без configurator потому что тут не только конфигуратор, но и транзитивная зависимость
DS
Так... я всё-таки за yii-event
ок, пусть так будет
АН
Может лучше yii-event-configuration?
АН
Сразу понятно что это
AM
Александр Некрасов
Может лучше yii-event-configuration?
По-моему, не надо тут -configuration
DS
без configurator потому что тут не только конфигуратор, но и транзитивная зависимость
что ты тут имеешь в виду под транзитивной зависимостью
AM
1. И без него ясно что эта штука даёт события для Yii.
2. Оставить место для манёвра.
2. Оставить место для манёвра.
AM
что ты тут имеешь в виду под транзитивной зависимостью
Что можно, по идее, не указывать event-dispatcher в пакете, который юзает yii-event.
DS
Что можно, по идее, не указывать event-dispatcher в пакете, который юзает yii-event.
А, так да
DS
По конфигуратору сначала пр посмотри, после него уже проще перенести
AM
Надеюсь, события туда не уйдут?
Нет. Это плохая идея.
AM
По конфигуратору сначала пр посмотри, после него уже проще перенести
Который?