Stefan Kanev

Update: I initially thought FVim does not support command mappings, because no mapping I tried work. Turns out - they work, but only if I enter them twice. That is,...

Няма тестове

> @skovachev Благодаря за труда! Подкрепям напълно коментарите на Гената и Станислав. TL;DR.

Изключително резервиран съм към дизайна, в който notification-и се създават в `before_create`. Lifecycle event-ите на `ActiveRecord` имат един тон проблеми, когато опира до ползването им за бизнес логика. Може да...

Чини ми се, също така, че много от тия работи трябва да стават в background-а. Създаване на 100 notification-а за 100 потребителя, например.

> Може би Service Layerish обекти, които създават запис и нотификация или малко trickery около Notification.send_notifications_for, примерно да приема блокче, което изпълнява преди пращането на съообщенията. Общо взето нови обекти...

> На първо четене, over-engineering. С какво сравняваме? Не даваш алтернативи. Това превръща "аргумента" ти в "не-аргумент". Следва подобаващ отговор. > Наистина ли имаме нужда от този слой? Да. >...

> Може да изчакаме с тази система, за вторият или третият път когато ни потрябва подобно нещо и имаме по-ясно видимост за проблема, които ще реши. Нямам нищо против да...

BTW, вчера спорих, с някой дето не беше чел кода, но имаше мнение за него. Наложи се да му покажа скеч на идеята ми: https://gist.github.com/skanev/b402f8a7554b1162d447 Ето предимствата: 1. Групира concern-а...

Ето още малко храна (за размисъл). Две неща са. За първото, как да е, но за второто се изненадвам, че аз трябваше да се сетя, понеже ми се струва, че...