Fork me on GitHub
#fulcro
<
2018-03-01
>
levitanong05:03:22

@wilkerlucio Any idea what it means when

[fulcro.client.impl.application] Mutation  fulcro.inspect.ui.transactions/add-tx  failed with exception Error: Assert failed: Path [:fulcro.inspect.ui.transactions/tx-list-id [:fulcro.inspect.core/app-id some-app.ui.root/Root] :fulcro.inspect.ui.transactions/tx-list] for append must target an app-state vector.
(vector? (get-in state data-path))

wilkerlucio17:03:23

I saw that a few times, are in the latest version?

levitanong18:03:46

Yes, it’s in the latest version.

wilkerlucio18:03:11

ok, can you open an issue on inspect with a minimum case, it's easy to reproduce?

magra14:03:24

I have the same message three times on every browser refresh.

wilkerlucio20:03:30

@U15BH4U4V I just sent a new snapshot that might fix it, please check the details on the link sent by @U0BR5D7A6

magra21:03:04

Thank you @wilkerlucio! The error is gone!😄

wilkerlucio21:03:44

glad to hear 🙂

levitanong17:03:12

That’s odd, doesn’t seem to have fixed it on my end.

levitanong18:03:22

woops, silly me. somehow it was showing up twice in my dependencies, and the other wasn’t updated.

levitanong18:03:25

it works now!

tony.kay05:03:46

FYI: Anyone wanting to try out cljs 1.10 I’ve just pushed 2.3.1-SNAPSHOT to clojars. There were two minor changes needed for it to work. I’ve not done extensive testing, but I do a formal release as soon as I get some reports of larger applications working.

myguidingstar16:03:25

@wilkerlucio how do I access to top level AST from my outermost wrap-parser plugin?

myguidingstar16:03:52

I want a plugin that read top level AST, calculate a value from the AST and inject to env so the all reader plugins can access the value

wilkerlucio17:03:32

@myguidingstar given this is the wrap-parser signature: (fn [parser] (fn [env tx])), the tx is your full query, so you can call query->ast on it to get the full source ast

myguidingstar17:03:11

I know, but I guess that's duplication

myguidingstar17:03:34

anything else to achieve the same goal?

wilkerlucio17:03:37

not that I know, because the parser make this internally, currently it is kind wasterful, it calls query->ast on every recursive call, but for server processing thats usually not a problem

myguidingstar17:03:26

I see, so one more is no big deal 🙂

myguidingstar17:03:52

where is query->ast?

wilkerlucio17:03:02

fulcro primitives

myguidingstar17:03:48

ok, I thought we've decoupled pathom from fulcro

wilkerlucio17:03:19

not yet, but it might be in future

myguidingstar17:03:34

thank you very much