This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2019-08-22
Channels
- # announcements (9)
- # beginners (96)
- # calva (14)
- # cider (49)
- # clj-kondo (29)
- # cljsrn (5)
- # clojure (55)
- # clojure-dev (37)
- # clojure-europe (4)
- # clojure-italy (4)
- # clojure-nl (5)
- # clojure-spec (5)
- # clojure-switzerland (3)
- # clojure-uk (5)
- # clojurescript (172)
- # cursive (14)
- # datomic (3)
- # duct (1)
- # emacs (6)
- # fulcro (17)
- # jobs (1)
- # leiningen (6)
- # nrepl (11)
- # off-topic (26)
- # pedestal (2)
- # re-frame (20)
- # reagent (9)
- # remote-jobs (3)
- # shadow-cljs (67)
- # spacemacs (24)
- # specter (6)
- # test-check (3)
- # tools-deps (18)
Fulcro 3 remotes are just maps…you could easily write one to use the f2 pathom remote @souenzzo
I should probably make a function that does so…it’s a generic transform from one interface to another
Ah, but it will bring f2 onto your classpath I guess, so it would need to be a separate lib or something
which i’m not going to do 😜 Basically f3 remote is a map with :transmit!
key that is a function that gets a send node (a map). See the implementations in F3. Just adapt that to proxy over to the f2 remote’s send method, and pass callbacks that adapt the response.
I implemented it
https://github.com/souenzzo/fulcro-pathom-realworld-example-app/blob/feature/fulcro/src/main/realworld_fulcro/proxy.cljc#L107
But I'm a bit unsure about this impl. the :com.wsscode.pathom/trace
isn't working on inspect
Need some docs about which keys we should pass to result-handler
It's really hard to find in the "main" impl with keys are passed to result-handler
https://github.com/fulcrologic/fulcro/blob/develop/src/main/com/fulcrologic/fulcro/networking/http_remote.cljs#L288
inspect will need upgrade, since it has to detect network type and augment it (so it can transform the query to add the trace property to the request)
oh, maybe not, this may need to go in fulcro code since the wrappers live in fulcro now
@U0CKQ19AQ I'm remembering now that this changed right, there is no more wrappers? sorry @souenzzo, I'm not close enough to this code to give you proper guide, would have to look again and understand the new integration better