rberger 2021-06-22T02:12:29.202Z

The re-frame-10x UI freezes when I reload the browser page, I don’t remember it doing it before upgrading to the latest versions of re-frame (1.2.0) and re-frame10x (1.1.4), though its possible its due to some other issue… If I clear out the reframe-10x items in LocalStorage, it creates a new 10x UI. But I wonder what I might be doing wrong that this behavior is happening. Using shadow-cljs 2.14.5. I don’t have a literal main but I do have:

:builds {:app {:target     :browser
                :output-dir "resources/public/js/compiled"
                :asset-path "/js/compiled"
                :modules    {:app {:init-fn visx.dashboard.core/init}}
                :devtools {:http-root "resources/public"
                           :http-port 8280
                           :preloads []}
                  {re-frame.trace.trace-enabled?        true

rberger 2021-06-22T02:13:38.203200Z

Not sure if it might be related to the shadow-cljs config. The instructions mentions module not modules or might there be something I need to put in the init code for some shadow-cljs lifecycle?

zackteo 2021-06-22T03:15:27.203400Z

@p-himik Strange, am certain my geoJSON is correct - but I do have this weird situation where the app won't "compile" until I save. But anyhow, I got it working! πŸ™‚ I do however have some questions, 1. ^{:key @state} , how might I know I would need to do something like this? Is there documentation that I can look at ? o: 2. I tried using reset! to change my geoJSON from a small square (geojson) on the left to a giant triangle on the right (other-geojson), I don't quite understand why the map will only re-render if I first change it to nil is this something to do with what triggers the re-render?

p-himik 2021-06-22T04:47:36.203900Z

Regarding GeoJSON being incorrect - it very well may be that it were simply my own experiments that were incorrect. And during my prior experience with GeoJSON, I always used feature and feature collections as the top-level objects. But seems like the spec allows having the geometry at the top level as well, so all is good. 1. When you use the same @state in a :div and in some component, and the latter is updated but the former is not. It means that the component has some issues in its state management or it simply disallows changing at least some of its state for some reason. Providing the :key metadata (or via a regular attribute, if the component itself supports it, like all the plain HTML entities) makes sure that when that key is different, the whole component is re-rendered. One thing to note - that key is stringified when it's used. 2. I blame it on the GeoJSON component's implementation. It has nothing to do with Reagent.

zackteo 2021-06-22T04:56:58.204100Z

Is the :key metadata is a HTML thing? or is that regular attribute ? Is there a link I can read up on this? :o I guess there's why in some example they wrap react-leaflet with some of their own structure

zackteo 2021-06-22T05:10:46.204500Z

Right there's this . But what is a regular attribute ? :o

p-himik 2021-06-22T05:13:20.204900Z

It's a React thing. Despite most of the documentation mentioning lists of all sorts, it's applicable outside of lists as well. β€’ β€’ Planwise doesn't use react-leaflet. It's the problem with react-leaflet, not with leaflet itself. There are two ways to use :key in Reagent:

^{:key something} [:div {:class ...} ...]
[:div {:key something, :class ...} ...]
I'm not 100% sure whether the latter works if the component doesn't handle :key explicitly. I just stick to the metadata way at all times.

thheller 2021-06-22T06:00:34.205800Z

config looks fine. it is :modules, where each entry in that maps defines a module

zackteo 2021-06-22T06:16:31.206Z

Okay! Thanks for all your help!! πŸ˜„ Appreciate it

superstructor 2021-06-22T09:13:20.207200Z

That shouldn't happen @rberger. Do you have a minimal reproduction ?

rberger 2021-06-22T17:13:27.207300Z

Let me see if I can reproduce it outside our relatively huge and not public webapp. Its consistently failing there of course :-)


I need to send a few dozen requests to the server, but I would not like to do it at once, but divide it into packets How can I e.g send e.g. 4 simultaneous requests and the next one only after all the previous ones have succeeded. How can I do this with re-frame-http-fx? I tried using async-flow, but it seems overly complicated in this case and maybe there is a simpler way?

p-himik 2021-06-23T09:10:54.226600Z

> implementing your own effect handler is somewhat tedious Not at all, given that it will be a thin wrapper around :http-xhrio. You don't have to reimplement the whole wheel, just add a couple of atoms to track the queued requests and the in-flight requests, and that's it.


@emccue I have currently created a similar ugly creation to yours, but I don't like it at all. As I mentioned, I tried to simplify it by using async-flow, but it made things even more complicated. Rewriting http-xhrio in my own fashion seems cool and minimizes the amount of code I need to write, because it's only a few lines

p-himik 2021-06-22T19:52:07.208700Z

I would probably create a new effect handler, similar to :dispatch-debounce.


Unfortunately, I can't quite picture it. After sending a request via :http-xhrio the next queued events are immediately processed, including the next :http-xhrio. I would have to throw calback to :on-success, or make some kind of counter and call the event in a loop until the counter is released


Am I thinking right?

p-himik 2021-06-22T20:00:57.209400Z

No. You would create your own version of :http-xhrio that's based on that but has rate limiting added on top.


ah, I misread, now it's clear


thanks! I'll try it this way


In general, this is another time when 5 minutes don't pass between my question and your answer. On the re-frame and reagent channel you are a one man army serially solving other people's problems. Appreciation. πŸ™ƒ

superstructor 2021-06-22T21:57:01.210500Z

Try 1.1.7 please. It fixes several bugs. @rberger

superstructor 2021-06-22T21:57:13.210700Z

(1.1.7 of re-frame-10x)