These are chat archives for dry-rb/chat

2nd
Feb 2017
Maciej Mensfeld
@mensfeld
Feb 02 2017 08:02
Tim Riley
@timriley
Feb 02 2017 08:06
@mensfeld things don't go to the website right away :)
Andy Holland
@AMHOL
Feb 02 2017 09:42
@solnic awesome :)
Piotr Solnica
@solnic
Feb 02 2017 09:49
I’ll build a website with docs once it’s out of alpha
which should be soon, because it’s so simple
the biggest challenge is to make this 1st class concept in rom
for now I monkeypatched SQL::Relation :monkey_face:
Tim Riley
@timriley
Feb 02 2017 09:53
"OH solnic now I monkeypatched..."
Piotr Solnica
@solnic
Feb 02 2017 09:54
heh btw THIS is how monkeypatching is useful
otherwise we’d be blocked
and in such cases I love Ruby SO MUCH
Tim Riley
@timriley
Feb 02 2017 09:55
Yeah I was just gonna say that too, haha :)
Piotr Solnica
@solnic
Feb 02 2017 10:02
btw instrumentation integration between Sequel/rom-sql and AS::Notifications for rom-rails == a monkey-patch
which I actually copied from sequel-rails railite
but it won’t be like that for long
because I’ll be adding proper support for instrumentation to rom and rom-sql (as I mentioned already)
Oskar Szrajer
@gotar
Feb 02 2017 21:03
Wot so fast ;] I need update again :)
@timriley I update dry view in the project and everything work ok (but will probably change few things after I see how you guys resolve it in berg, I'm not fully happy how I extend view_context by assets and current user . Plus I spend some time investigating why my partial do not work (single one) and found that it have the same name as one method in context. So it render method not partial, It's something a lot probably hit one day, so maybe another idea here will be better to explicity know when I want to render partial and when method
Oskar Szrajer
@gotar
Feb 02 2017 21:33
@solnic I think your last update for dry-web-(roda) can cause problems - (need changes in dry-web-roda). It requires log directory to exists for any app in apps folder, store logs there not in main log directory, plus app generator for dry-web-roda already add to boot logger and register it as logger in Container so it crash . Maybe it's only my app but worth to check
Tim Riley
@timriley
Feb 02 2017 21:35
Thanks for for trying these out, @gotar. I'll check dry-web-roda today
Oskar Szrajer
@gotar
Feb 02 2017 21:37
chages required (I assume are required) for dry-monitor are super easy, both for app generator to drop custom logger and add two directories