Fork me on GitHub
#untangled
<
2017-06-13
>
cjmurphy01:06:11

@tony.kay: An easier read for first timers than previous documentation because there are no longer gaps. I really like it. If you needed to shorten it the om/computed in depth explanation could be put elsewhere - for getting started all application programmers need to know is that functions passed down to child components are to be handled differently: you can get a long way without knowing exactly why that's the case. That's my proofreading feedback.

tony.kay01:06:36

I can’t get away from computed, because of thecallback I need 😕

tony.kay01:06:55

It would be confusing to use it without some kind of nod to why

tony.kay01:06:29

or did you mean to just kind of trim it down?

tony.kay01:06:05

I guess that is what you’re saying….sorry didn’t read carefully enough

tony.kay01:06:05

Thanks for proofing it! I think the new additions also make it seem more approachable. No parsers needed anywhere, until you start wanting to convert queries to graph walks of SQL

tony.kay01:06:44

and even that could be done as a transform instead of a parse

cjmurphy01:06:09

What is great about it is the flow - telling a story and filling missing bits in as go.

tony.kay01:06:41

Yeah, I had tried to do that in the quick tour in the dev guide, but I think it was too light

tony.kay01:06:54

not sure how many times I have to write this before I like it 😜

tony.kay01:06:35

3000 lines of code, 20,000 lines of docs

cjmurphy01:06:04

The first time users needs to go away having understood all, or almost all. The computed stuff is difficult and not absolutely required.

cjmurphy01:06:57

Esp - 'wrong way' - I must admit I didn't read that part.

tony.kay01:06:13

Thing is I’ve just spent several pages telling that you “just pass stuff as props”. I’ll re-read it and see if I can make it shorter without feeling like I’m going to lose people

tony.kay01:06:25

There’s a fine balance between boredom and confusion

tony.kay01:06:35

and unfortunately that line is different for everyone

tony.kay01:06:47

I’d rather have more bored and less confused

cjmurphy01:06:52

I've never really 'got' what's going on with computed, so perhaps it is hard for others too. All I know is as an app programmer I follow the rule - "if its not props I use computed".

tony.kay01:06:18

perhaps if you read “wrong way” part…

tony.kay01:06:32

but I hear you, rules are easy to follow

tony.kay01:06:37

you don’t need to know why

cjmurphy01:06:38

Yes I will now 🙂

tony.kay01:06:42

I get stuck on the why

tony.kay01:06:49

I don’t like magic

cjmurphy01:06:04

But the line is important - between what user of framework needs to know and what builder of the framework needs to know.

tony.kay01:06:47

agreed. Understanding computed is one I think the user should know..even though I see your point that there is a rule to follow. Cause if you don’t follow the rule, it looks like it works…until it doesn’t

tony.kay01:06:12

and a lot of devs will run with things that look like they work, then get bogged down when they do “weird things”

tony.kay01:06:57

and there’s nothing I dislike more that banging my head against a wall when trying to get something done. I guess that’s why I want to knw why…helps me debug

tony.kay01:06:11

Perhaps I’ll move the why to a section marked optional, and simplify the main flow to just state the rule…“if you want to make stuff up that didn’t come from the query engine and pass it to a child, put it in computed”

tony.kay01:06:52

i just re-read it. I reworded one sentence, but otherwise I still like it as-is

tony.kay01:06:11

the explanation is pretty simple, if anything I could make a diagram or something.

tony.kay01:06:35

I reworded this:

The problem is that Om can optimize away a re-render
of a parent when it can figure out how to pull just the data of the child on a refresh, and in that case the
callback will get lost because only the database data will get supplied to the child!

tony.kay01:06:29

A few more tweaks and now I’m definitely happier with it. Thanks for the feedback

mitchelkuijpers14:06:51

I have an ident that points to a ident that I use in a form that then breaks on untangled.ui.forms/validate-form Are idents that point to idents supported in om.next or am I breaking stuff? Or maybe I found a issue on form-support in untangled

tony.kay20:06:57

An ident is supposed to point to a table entry

tony.kay20:06:09

I don’t think a talble entry is supposed to be anything other than a map

tony.kay20:06:18

you’d have to look at the implementation of db->tree in Om, but I suspect it isn’t