Fork me on GitHub
#om
<
2017-08-29
>
linuss14:08:04

Hey guys, I have the following bits of code and data: https://pastebin.com/uShRBusM . I'm trying to find a way to include a components property as a query parameter in an ident (in order to resolve foreign keys). Does anybody have an idea how to do this?

rplevy17:08:12

Does anyone know what ref-cursor is for? The documentation was unclear to me. I found that I could remove it from an Om application with no loss of functionality. Having it in, on the other hand was causing performance problems because it was being called very frequently in the context where it was used.

rplevy18:08:27

I'm trying to test for problems with stale data from cursor as that is my best understanding of what ref-cursor is for.

drcode19:08:30

When I do (om/set-query! this {:query ...}) will it affect all instances of the component or only the this component?

driphter22:08:20

@anmonteiro Is your omify library currently working/usable?

anmonteiro22:08:19

@driphter haven’t tried it in a while

anmonteiro22:08:26

report any issues you find

driphter22:08:42

Alright, will do. Thanks!

rplevy23:08:01

I've been testing leaving out ref-cursor and I don't find any problems so far with doing so. I'm looking for someone who understands om internals a little better to tell me if it's a terrible idea. 🙂

petterik23:08:02

Is there a reason why react16 wouldn't re-render with om.next? I've heard that the isMounted code would have to change, so I changed that: https://github.com/petterik/om/commit/9735bdc362f991930c43962df66baf3bbcc3130a#diff-ee424e6559581b5b893371373a777e5eR193 Anything else? @anmonteiro do you know?