search

cjohansen / portfolio

Eclipse Public License 1.0
222 stars 13 forks source link

The required namespace "snabbdom" is not available #4

Closed BorisKourt closed 1 year ago

BorisKourt commented 1 year ago

I understand that this is related to: https://github.com/cjohansen/dumdom/issues/23

But I am not sure what the current workaround is. I am using shadow-cljs and the rest of the application is in reagent. I am assuming that dumdom is used for rendering the portfolio UI itself?

cjohansen commented 1 year ago

Yes, that's right. I'm working on a fix for this. I'm off for the weekend, but this should be handled early next week at the latest.

cjohansen commented 1 year ago

I'm not familiar with shadow-cljs, but if you can try running portfolio from git with the sha 4e25bcda2f8577b6ea863c2af892c7217a3dc94a, it should hopefully work.

BorisKourt commented 1 year ago

I had to use 6beb9521ef5b0f5a631fe6979530f99915730c51 as deps.edn couldn't resolve the sha above.

Encountering new errors in my current project, will try to isolate in a repro.

cjohansen commented 1 year ago

I got it working. A fix is available in 2023.03.28 (...). Make sure you configure shadow correctly, as per https://github.com/cjohansen/portfolio#shadow-cljs

BorisKourt commented 1 year ago

It works, thanks.

If someone reads this that also has shadow-cljs run separately from their back-end launched via lein: add classpath:public to the :resource-paths vector of your project.clj. And add the dependency to Portfolio to project.clj.

cjohansen commented 1 year ago

Feel free to submit a PR on the Readme if there are more ways to configure this 👍