Fork me on GitHub
#fulcro
<
2017-08-27
>
tony.kay03:08:15

@wilkerlucio Hm. That sounds like a bug to me.

tony.kay03:08:28

Do you see the UI property go over the wire? Hm..actually, the not-found markers are supposed to be removed after the network interactions…they should not be merged into the app state.

tony.kay03:08:58

or rather, after they are merged into app state, there is a post-step that is supposed to remove them. Have you augmented anything…like merge?

wilkerlucio13:08:43

@tony.kay when I mentioned I saw it, wasn't in my state output, was debugging the merge function, the merge-ident function is seeing :not-found and by that is removing it from my app state

wilkerlucio14:08:03

looks like the ui property got correctly removed at network send time, but at same time, when doing the diff to compare the query with the network result, the ui is being taking in consideration (and it should not)

tony.kay18:08:18

agreed. I’ll open an issue

tony.kay18:08:21

@wilkerlucio Can you make a minimum failing case? I just modified the load samples demo to have ui attributes and an ident-based refresh, and it works fine. I also double-checked the tests, and the cover this pretty heavily, so you’ve possibly found some kind of corner case.

tony.kay19:08:31

If you check out that branch and can get that demo to fail in some way, I could diagnose it further. I’m triggering loads based on ident and top-level to-many…and both preserve the ui attributes in state. The commit link shows the changes I made to the demo.

tony.kay21:08:03

Fulcro-SQL is alive! I just got many-to-many joins across a standard join table working. So, graph queries work for the following standard SQL schema scenarios: - scala values on tables - Joins: - to-one, with FK on source table - to-one, with FK on target table - to-many, with FK on source or target table - many-to-many with join-table that holds the FKs for the two related tables (points to their PKs) 0.0.1-SNAPSHOT is on clojars with this update.

tony.kay23:08:51

Fulcro-SQL pushed again. I added basic support for MySQL/MariaDB. I’m not a MySQL guy, so I’m not certain about what I did, but it basically tests out.

tony.kay23:08:20

I’ve released version 0.0.1 of fulcro-sql to clojars.