Hey everyone! Was wondering if anyone had a writeup/documentation about porting a lein cljs project with project.clj over to shadow-cljs?

no write-up but have done it a couple of times if you have any questions

Did anything happen to the shadow-cljs nrepl? I got some nullpointer exception whenever I am trying to make autocompletion with Cider

It works with 2.9.10 but not 2.10.1

https://github.com/rksm/clj-suitable/issues/15

stay with 2.9.10 if you want cider auto complete for now. not sure I can or want to restore the not-public API it used before.

Thanks. So clj-suitable can actually change their API and it would work?

that is what I'm trying to determine in the ticket. not actually sure what they need. I think the new API fn should cover everything.

I'll see if I can restore the old stuff in some way but not sure I can. probably best to stay with 2.9.10 if you care about cider autocomplete

Thanks 🙂

Hi I am trying to use this React component: https://github.com/christo-pr/dangerously-set-html-content , I am using Reagent. At the top I have:

(ns app.core
  (:require ["dangerously-set-html-content" :default InnerHTML]))

(defn component []
  (let [html-str "blabla"]
      [:> InnerHTML {:html html-str}]))

React.createElement: type is invalid -- expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it's defined in, or you might have mixed up default and named imports.

@luca.cambiaghi try

(ns app.core
  (:require ["dangerously-set-html-content" :as InnerHTML]))

@thheller awesome, that works! I feel stupid for not having tried it before! Thank you!!

Every time I start shadow-cljs up, it sends a build error on this particular line:

50 |        #::table
----------------------^---------------------------------------------------------
No namespace: compute.ui.components.table found

If I go make a small change to the table ns, the error goes away. It will then find another place where the namespace map syntax is used and fail on that. For example

139 |        #::calendar{:value     selected-date
-------------------------^------------------------------------------------------
No namespace: compute.ui.components.calendar found

this is not an error from shadow-cljs. maybe from the reader?

Trying the latest version. Was on 2.9.8. Oh, yeah probably the reader.

2.10.2 fixes it. Weird.

Some sort of bug in the reader in between those versions. A bit surprising.

there have been no tools.reader changes in those versions

check your classpath. maybe you have some conflicts?

Oh, perhaps. Will check. Latest shadow-cljs will print this warning to the console on reload: couldn't find custom :build-notify

whoops

will be fixed soon. looking into something else before I make a new release

Hello all, I´ll deploy my first shadow-cljs project. I know that I´ll have to compile it with npx shadow-cljs realease app . But how do I pack the node_modules?

My node_modules has 270 Mb, do I have to copy it to my server?

assuming you are talking about a :browser build you only need the files in the :output-dir

yes, :browser . If I did something like <link rel="stylesheet" href="node_modules/suneditor/dist/css/suneditor.min.css"> , copy it to the css asset?

there is cljs-runtime, do I have to copy it too?

if you are referring to css files directly then you need to copy those

the cljs-runtime dir is from dev builds (ie. watch). it is not produced or required by release builds.

so, only app.js?

I don't know your :modules config. if you only have :modules {:app ...} then yes

:modules {:app {:entries [app.core]}}

so it will compress all the dependencies in one app.js ?

yeah that produces a app.js in your :output-dir

yes

Good

thank you !