Fork me on GitHub
#re-frame
<
2018-03-28
>
danielcompton00:03:25

@cjsauer oops, not without attaching a debugger and calling it from console

MegaMatt00:03:35

hi all, I've done my first clojure project in re-frame. I rewrote an angular project just to get experience. I was hoping that someone would be willing to code review it, preferably someone good, opinionated, and rude is perfectly fine.

MegaMatt00:03:47

BTW. the project was rewritten from angular to angular +nrgx and went from 5k loc to 2.5k loc then the rewrite to re-frame brought it to > 1k with the hiccup (2.5k didn't include html)

MegaMatt00:03:03

very happy with using re-frame

MegaMatt00:03:59

If anyone is interested PM me, I can also send a little litecoin your way for the help.

mkvlr07:03:44

is there a way to ensure ordering in effects?

alex-dixon15:03:05

What’s the most idiomatic way to issue an ajax call when a route changes (or is about to change?) to a particular route?

alex-dixon15:03:23

I’m thinking of having event handlers for e.g. [:route-changed :home] [:route-changed :about]. Both require different ajax calls. Should I instead have one event handler for [:route-changed] and look up the ajax call based on the key?

alex-dixon15:03:35

I think I’m aiming for something like (reg-event-fx [:route-change :home] ...) but not sure there’s a way to do this that would short circuit (not call the handler) when the route change event argument isn’t :home. Is there?

hoopes23:03:40

WARNING: Use of undeclared Var cljs.tools.reader.reader-types/reader-error at line 9 resources/public/js/compiled/out/rewrite_clj/reader.cljs                   
WARNING: Wrong number of args (2) passed to cljs.tools.reader/read-token at line 182 resources/public/js/compiled/out/rewrite_clj/reader.cljs
When i turn on day8.re-frame-10x.preload in my project.clj (in :preloads), I see the above warnings. When I remove it, the warnings go away. Should I be concerned about these, or are they a known issue? If not, what's the best way to find out what's happening? Thanks! I started from the re-frame template with +aliases +10x.

danielcompton23:03:12

Open an issue, that doesn't sound right