yii2-attachments icon indicating copy to clipboard operation
yii2-attachments copied to clipboard

#69 Add event after attach files

Open sokollondon opened this issue 6 years ago • 4 comments

sokollondon avatar Jun 14 '18 11:06 sokollondon

@sokollondon Hello, thanks for your PR. Can you please separate it to one PR per issue?

drtsb avatar Jul 28 '20 19:07 drtsb

@drtsb Сергей, не знаю как разделить сейчас. Ведь все 3 коминта в одной ветке. Зачем это нужно?

sokollondon avatar Jul 31 '20 08:07 sokollondon

@sokollondon приветствую! К #69 относится только 1 коммит, рекомендую создавать отдельную ветку для каждого PR. Сейчас можно стартануть новую ветку от текущего мастера и с помощью cherry-pick взять один коммит туда, после чего создать новый PR.

drtsb avatar Jul 31 '20 19:07 drtsb

@drtsb почему бы не применить весь PR? Все три коммита полезные и не уменьшают производительность. #78 правка опциональная. Кому не требуются эти Enh могут не использовать)

sokollondon avatar Aug 04 '20 04:08 sokollondon