Hey! Yeah, I did have vim-fireplace + clojurescript working with piggieback before. Now I'm using conjure instead of vim-fireplace, but when I was using vim-fireplace + figwheel, I found this guide to be the most helpful: https://figwheel.org/docs/vim.html Are you using figwheel/shadow-cljs or something else?

@UEJ5FMR6K I'm using neither figwheel or shadow-cljs (using lein-cljsbuild atm for building cljs resources). I've been working through Web Development with Clojure (from Pragmatic bookshelf) when I hit this snag with with vim-fireplace/cljs. I do see a chapter or so later in the book there is some set up for shadow-cljs and from looking at some online articles may find luck there

Thanks for this figwheel resource too 🙂 Will report back on progress

use [...] instead of (...) when working with components.

https://github.com/reagent-project/reagent/blob/master/doc/UsingSquareBracketsInsteadOfParens.md

Okay, done that, not solved the issue though 😞

Can post my subs if thhat helps

Sure.

Done, thanks for your help

You mean, you fixed the issue?

Ah, I see. Should've posted them in the thread.

I can't see anything obviously wrong. At this point, unless someone else sees something, I need a proper minimal reproducible example.

Nah not fixed the issue, and i thought the thread window might be too small. I can move them

Better move them, yeah. The thread can be expanded to the full width.

(rf/reg-sub
 :assets
 (fn [db _]
   (:assets db)))

(rf/reg-sub
 :active-server-id
 (fn [db _]
   (:active-server-id db)))

(rf/reg-sub
 :servers
 :<- [:assets]
 (fn [assets _]
   (filter asset/server? assets)))

(rf/reg-sub
 :clients
 :<- [:assets]
 (fn [assets _]
   (filter asset/client? assets)))

(rf/reg-sub
 :active-server
 :<- [:active-server-id]
 :<- [:servers]
 (fn [[active-server-id servers] _]
   (first (filter #(= active-server-id (:_id %1)) servers))))

(rf/reg-sub
 :available-clients
 :<- [:clients]
 (fn [clients _]
   (filter #(=(count (:peers %1)) 0) clients)))

(rf/reg-sub
 :server-peers
 :<- [:clients]
 (fn [clients [_ server]]
   (filter #(asset/has-peer? server %1) clients)))

If i linked all the code would that help? If that's too much work to look through then no worries

Just create a small GitHub repo that I can clone and then run with a single command.

Okay will try add some data seeding

Here you go

https://github.com/Cgboal/wire-runner

You'll need mongo db but it'll seed itself

docker run --name=WirerunnerMongo -v /var/lib/WirerunnerMongo:/data/db -p 27017:27017 -d mongo

then just lein run and lein figwheel

Go onto the web app, click add peer, select one from the available clients, it adds on the backend, the app-db updates, but the tables don't get re-rendered

Well, that's not exactly minimal. :) But either way, I cannot run MongoDB: "Error: error checking path "/var/lib/WirerunnerMongo": stat /var/lib/WirerunnerMongo: no such file or directory" I'm using podman instead of docker, but that shouldn't matter.

You can get rid of that argument if you like, it's just for container persistence

Which you wont need, just pasted the docker cmd i used to start it

Don't get me wrong - I want to help, but I really don't want to figure out how to run your project that's very far from a minimal reproducible example. Now I'm getting HTTP 404.

On http://127.0.0.1:3000/ ?

Figwheel: Starting server at

Did you do lein run, if so, it's 127.0.0.1:3000

OK, that seems to work.

Nice one, thank you for taking the time to help

There's a bunch of JS errors. Potentially, then can be the ones that cause your issues. Have you tried fixing them?

Only ones i was getting was Warning: Each child in a list should have a unique "key" prop.

You mean that?

And <th> cannot appear as a child of <thead>.

Fixed that in one place but missed the other, done that now, not affecting it, will look at the warning

Docs say the warning just pertains to performance of updates

Not sure what docs you're looking at, but that's not correct.

Ah, wait - I'm probably mixing it up with index keys.

Fixed the keys issue, problem persists

It was re-rendering properly before i added the subscription to peers-table

Fixed it. This was the solution

(rf/reg-event-db
 :swap-asset
 (fn [db [_ asset]]
   (update-in db [:assets]
          (fn [assets asset]
            (conj
             (filter #(not= (:_id asset) (:_id %)) assets)
             asset))
          asset)))

(rf/reg-event-db
 :swap-asset
 (fn [db [_ asset]]
   (update-in db [:assets]
          (fn [db asset]
            (conj
             (filter #(not= (:_id asset) (:_id %)) (:assets db))
             asset))
          asset)))

Thank you for helping me narrow down the issue, so many moving parts and i wasn't sure which bit i could have been fucking up

mhh interesting. I thought it would be something like a reference

FWIW: I am trying to use leiningen for dep management instead of the deps.edn that the Getting Started suggests

I remember running into problems with dependencies

because node versions changed how commonJS works, to incude outside modules

so first, you might want to look at your node version, and commonJS version

I just upgraded to latest node LTS. Is commonJS a part of node?

it’s a dependency module library, node sucked at including outside js, in the early days

can you verify that you have commonJS in your package.json?

hm fair, fwiw it works to use deps.edn and the getting started launching with clj

I don’t even have a package.json 😅

really? this is a node project? you are compiling to CLJS?

my understanding was that cljsbuild would compile my .cljs to a .js which I could then serve directly with node. I might be completely off though

cljs, will convert to js, yes

but I really think you need your commonJS, package.json, 90% sure

or you can’t include dependencies

it’s an educated guess

hmm, but wait

it really looks like ‘goog’ isn’t made available inside the base.js module

one sec

https://github.com/raml-org/datatype-expansion/issues/72

goog is the google-clojure library… it looks like

npm install google-closure-library

I think you are missing it… and base.js wants it installed

I did npm install goog earlier as well

ok, then you should have a package.json file

I would def stick to how node setups the project. with package.json

I just got the package-lock.json, but I now created a node project as well

the weird thing is that I know have a frankenstein of a leiningen and node project in the same folder

yep… I can screen share… if you want

ah no need. But thanks:) I will poke a bit more.

things are bit more calm on Fridays

if it's calling goog it's not compiling to node, it's compiling to the browser

https://figwheel.org/docs/nodejs.html

I don't know for sure but I really doubt that. TS can consume JS and CLJS produces JS - there should be no problems.