Fork me on GitHub
#graphql
<
2017-06-28
>
jonpither11:06:11

Hi 🙂 Wondering why I can't use normally defined objects as argument types, rather I must define input-objects in the input-objects section

domkm15:06:29

@jonpither Inputs cannot be cyclic while outputs (normal objects) can. GraphQL makes this explicit by requiring entirely different types.

bbloom16:06:18

@hlship i was wondering about the global exception handling strategy that exists there now - is there discussion of the change somewhere in particular i should look at?

hlship16:06:48

Just the PR and related issue, plus my own experience developing with Lacinia ... the 0.18.0 behavior makes it hard to track down where failures occur, since they get swallowed by Lacinia. The unreleased behavior encourages you to catch exceptions inside your non-trivial field resolvers and then use resolve-as.

bbloom16:06:33

that seems sensible to me - also because partial failures requires active work to support on the client

hlship16:06:07

Yes, this is just Lacinia getting out of the way of things it can't take care of perfectly, delegating the application code.

bbloom16:06:08

in order to debug issues, i had been swap!-ing a snapshot of the context in to a dummy var and then running resolvers by hand

bbloom16:06:45

not sure if this makes sense, but there may be an opportunity for some public api to enable individual resolver testing

bbloom16:06:51

besides, you know, just calling one

bbloom16:06:39

for me that was just capturing the actual arguments - maybe some function that retries a resolver given an error map - or maybe i’ll just see how it goes w/ the new exception behavior

bbloom16:06:04

anyway, so var lacinia has been very reliable. thank you!

hlship18:06:26

bbloom: I'm a real stickler for validating input: both the schema and the query. I think that results in fail early with a useful exception, and that leads towards stability.

hlship16:06:15

Great to hear that!

hlship16:06:03

The other devs are good at keeping my framework-itis in check. I'm learning.

bbloom16:06:26

reeessssiiiiist 🙂

matthewdaniel20:06:13

Would lacinia work compiling to clojurescript or is it stuck only on the jvm?

hlship21:06:54

Currently JVM only, because of dependency on Antlr4. I'd like to look into opening that up a little, but it's simply not a priority at our end, and other things are.