shared.js:1552 TypeError: Super expression must either be null or a function, not undefined
    at _inherits (control.js:26)
    at eval (control.js:50)
    at Object.shadow$provide.module$node_modules$react_leaflet_control$dist$control (control.js:49)
    at Object.shadow.js.jsRequire (js.js:66)
    at Object.shadow.js.require (js.js:113)
    at eval (quagga.components.dataview.map.view.js:4)
    at eval (<anonymous>)
    at Object.goog.globalEval (shared.js:486)
    at Object.env.evalLoad (shared.js:1549)
    at tabbed-dataview.js:250

the require looks like this

["react-leaflet-control" :default Control]

the package https://github.com/LiveBy/react-leaflet-control

my code looks something like this

[:<>
     [:> TileLayer {:url "https://{s}.}]
     [:> ZoomControl {:position "bottomleft"}]
     [:> Control [:button "click me"]]
     [:> FeatureGroup
      (map
        (fn [position] [:> Marker
                        {:position position,
                         :key position,
                         :eventHandlers {:click (fn [e]
                                                  (js/console.log
                                                    (-> e
                                                        .-target
                                                        .-_latlng)))}}])
        geopoint-positions)]]

I'm starting to think it's something that's wrong with the package that's the cause for this; maybe it's not compatible with shadow-cljs?

@USWTQB9RU I think the components TileLayer, ZoomControl etc need to be inside a Map component. For Example: This worked for me

(ns example.views
  (:require
   ["react-leaflet-control" :default Control]
   ["react-leaflet" :refer [Map TileLayer ZoomControl]]

(defn main-panel [style]
  [:div {:style style}
    [:> Map {:center [51.3 0.7] :zoom 10}
     [:> ZoomControl {:position "topright"}]
     [:> TileLayer {:url "http://{s}."
                    :maxZoom 18}]
     [:> Control {:position "topleft"}
       [:button "click me"]
       "Reset View"]]])

thanks @U02806HQX1C 🙂 . I actually did put them them inside a MapContainer component that wraps the component I pasted up here ☝️ . On the other hand, I later also found a workaround where I don't need to use this library. but I still am curious as to why I get the above error

Cool. I got similar error when I used the Control component in isolation. So, I thought that might be the issue.

no clue what expo.root is so can't say

It would be this https://github.com/PEZ/rn-rf-shadow/blob/master/src/main/expo/root.cljs

I’ve used [shadow.expo :as expo] from your reagent-expo example and the behaviour is same.

(ns 
  (:require [shadow.expo :as expo]
            [re-frame.core :as rf]
            ["react-native" :refer [View]]
            [reagent.core :as r]
            [example.screens.home :refer [home-screen]]))
  
(defn root []
  [:> View
   [home-screen]])

(defn start
  {:dev/after-load true}
  []
  (expo/render-root (r/as-element [root])))

(defn init []
  (rf/dispatch-sync [:initialize-db])
  (start))

Seems like an extension of this issue https://github.com/thheller/shadow-cljs/issues/544#issuecomment-521662073 Adding {:dev/after-load true} as suggested in the issue

;; app.cljs

(defn start
  {:dev/after-load true}
  []
  (expo/render-root (r/as-element [root {:x (js/Date.now)}]))) 

well did you try the suggestion with using an extra variable?

react or reagent stop rendering if they encounter a component that hasn't changed

you can set :devtools {:reload-strategy :full} in your build config to reload many more files than it usually would

slower overall but might fix you issue

but the issue is not with shadow-cljs. it is reagent/react deciding to stop to render

ie. when you only edit image.cljs then [root] didn't change. so reagent/react decide they can stop rendering completely and never get to the actual image which actually changed

> well did you try the suggestion with using an extra variable? You mean adding {:dev/after-load true} Yes I’ve tried this. Same behaviour.

> you can set `:devtools {:reload-strategy :full}` in your build config to reload many more files than it usually would This worked… 🙏

I meant this [root {:x (js/Date.now)}]

the :dev/after-load just tells shadow-cljs which function to call after its done reloading code. you need that regardless

the :x would be the "hack" to force reagent/react to actually render the component and not skip it

> I meant this `[root {:x (js/Date.now)}]` Oh right. My bad, sorry :face_palm: This worked.

without the full reload strategy.

Thanks for your prompt response and for Shadow as well. 🙌

The build config I'm using:

{:target :npm-module
 :output-dir "node_modules/"
 :entries [malli-ts.ts]}

It is compiling successfully but failing to import the module 😕

I just spent some time tangling with cljs->node, you might try wrapping the output from the compiler in this UMD wrapper: https://groups.google.com/g/clojurescript/c/vNTGZht1XhE

@U4U6BDQTE which shadow-cljs version? this should be fixed in the latest?

@U028BUU1P3R that code is ancient history and not relevant anymore

good to know 🙏

FWIW prefer using :node-library over :npm-module when you can

oh thanks!

I did switch to :node-library and it's working fine 🙂

shadow only watches package.json files for changes, so if you touch package.json it should update

This worked, thank you so much! 🎉