These are chat archives for dry-rb/chat

8th
Feb 2017
Tim Riley
@timriley
Feb 08 2017 04:18
New homepage is up, along with a bunch of other design tweaks! http://dry-rb.org
Piotr Solnica
@solnic
Feb 08 2017 05:28
:tada:
Adam Davies
@adz
Feb 08 2017 07:03
Just in time for tomorrow @timriley? :)
Looks good
Tim Riley
@timriley
Feb 08 2017 07:04
@adz Yep, nothing like in-person exposure to 100s of ppl to motivate you 😅
Adam Davies
@adz
Feb 08 2017 07:06
:sparkles: :clap: good luck!
Oskar Szrajer
@gotar
Feb 08 2017 07:32
:heart:
Sergey Kukunin
@Kukunin
Feb 08 2017 09:13
great update on homepage! documentation really matters
Andy Holland
@AMHOL
Feb 08 2017 09:17
:clap: Nice one @timriley, looks really good :)
Oskar Szrajer
@gotar
Feb 08 2017 09:21
@timriley about new sql rom instrumentation from dry-web roda, I tried to add it to my project but get error
    notifications.instrument(:sql, name: name.relation, query: dataset.sql) do
      super
    end

  NoMethodError:
    undefined method `instrument' for nil:NilClass
not sure I miss something in my code or it not working at all ;]
Oskar Szrajer
@gotar
Feb 08 2017 10:24
so the notification for rom is not set
Piotr Solnica
@solnic
Feb 08 2017 10:30
@gotar you need to set it up manually for now
Oskar Szrajer
@gotar
Feb 08 2017 10:41
yeah but I more mean dry-web-roda do not do it anywhere, or I miss it
so people will got error i think
Piotr Solnica
@solnic
Feb 08 2017 10:41
it does it when you generate a skeleton
it’ll put lib/rom/notifications.rb file
Oskar Szrajer
@gotar
Feb 08 2017 10:42
i just generate new one
and there is no lib/rom/notification file
there is lib/rom/instrumentation
but it's the file that cause error
Sergey Kukunin
@Kukunin
Feb 08 2017 17:20
I'd like to implement this and send PR: dry-rb/dry-logic#29. Please review and give your ideas
Piotr Solnica
@solnic
Feb 08 2017 21:32
@Kukunin sorry man for lack of replies, I’m at RubyConfAU so busy days. I’ll get back to you in a couple days
Sergey Kukunin
@Kukunin
Feb 08 2017 21:33
@solnic thanks. I appreciate your help. Good luck on the conf