Fork me on GitHub
#fulcro
<
2021-05-25
>
Jakub Holý (HolyJak)07:05:14

🙏 I have added a https://github.com/fulcro-community/fulcro-exercises/blob/master/src/holyjak/fulcro_exercises.cljs#L195 and would appreciate your help: 1. Beginners - is it clear and feasible? Anything to improve? 2. Experienced - is https://github.com/fulcro-community/fulcro-exercises/blob/master/src/holyjak/solutions_read_when_desperate.cljs#L195 OK-ish? (Read the exercise description under the first link first) Thank you!

🙌 9
Björn Ebbinghaus13:05:58

There is: com.fulcrologic.fulcro.algorithms.normalized-state/remove-entity Maybe show that instead of a delete-player Not really task related: L226: @(::app/state-atom app6)(app/current-state app6) L248: (->> players (map :ui/checked?) (every? boolean)) Why not (every? :ui/checked? players) ?

❤️ 3
Jakub Holý (HolyJak)17:05:57

Thanks a lot! Great point! I didn't know of remove-entity 🙏 To the last question: because I should not be working late in night :-)

Björn Ebbinghaus19:05:30

There are some useful functions I didn't know about for ages. Looking into the code today, I learned for the first time, that there is norm/integrate-ident and norm/remove-ident. I always used targeting/integrate-ident* and merge/remove-ident* . :-)

❤️ 3
Jakub Holý (HolyJak)19:05:31

FYI I pushed an update based on your suggestions. Thanks a lot for your help, much appreciated!

Jakub Holý (HolyJak)20:05:05

BTW I have also added a tip about the norm/* utils to the Minim. F. Tutorial, because these should be more widely known 🙂 Thanks for it!!!

3
Jakub Holý (HolyJak)16:05:27

@tony.kay Any plans to release Fulcro 3.4.22? There is the links from err/warn logs to the Guide, the CljDoc build fix, and few other small things 🙏

tony.kay16:05:45

ah, didn't realize I'd missed it

tony.kay16:05:46

sure

❤️ 3