Fork me on GitHub
Slack Archive

This page is not created by, affiliated with, or supported by Slack Technologies, Inc.

2020-04-24

Channels

#pathom
<
2020-04-24
>
jmayaalv12:04:05

hello, wanted to say that we have been evaluating pathom last few days and we are blown away. Serioulsy good job @wilkerlucio

jmayaalv12:04:24

and commiters

jmayaalv12:04:59

thank you for such a great tool

jmayaalv12:04:35

(cant’ go back to graphql now) 😞

👍 20
myguidingstar14:04:08

@jmayaalv it would be great if you can share your evaluation

👍 8
Jakub Holý (HolyJak)15:04:49

Hello! I struggle translating a working GraphQL query into Pathom Connect + graphql. I have described my problems here https://github.com/wilkerlucio/pathom/issues/155 and would appreciate any help. Thank you!!!

wilkerlucio17:04:27

thanks for the great report, I'll have some time to look into it later today

Jakub Holý (HolyJak)18:04:06

Thanks a lot! I plan to add a little more high level tests for the GraphQL PC that could also serve as a documentation (query + index -> resolver -> driver that returns hard-coded response -> errors and data). The current unit tests are too low for that.

Jakub Holý (HolyJak)20:04:48

@wilkerlucio Could you perhaps be so kind and show me how to write such a high-level/integration style? I see I know too little to be able to pull it off. Something like running 

(parser
  {::pc/indexes <based on graphql2-test/indexes>
   ::pcg/prefix "service"
   ::pcg/ident-map {...}
   ::p.http/driver (fn [r] (verify-expected-query r)
        "{\"data\":{...}}")}
   [{[:service.Customer/id "123"]
     [:service.Customer/cpf]}]
? I guess that graphql2-test/query-env quite close to the env I need, I would perhaps just need to plug into the env the actual com.wsscode.pathom.connect.graphql2/graphql-resolve 🙏

wilkerlucio21:04:45

for the integrations like these, I want to warn you they are about to change, I plan to release this weekend some of the new stuff, and sadly the graphql integration will need rework (graphql3 is coming 😛, but probably not this weekend), so I like to write about those integrations for this new scenario, since this is the thing moving forward

👍 8