Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tutorial for core.async #100

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Conversation

jamesnvc
Copy link
Contributor

You were mentioning before you were feeling a little overwhelmed with writing this series of tutorials and I wanted to try to lend a hand! I'm not sure if this is the sort of thing you'd be interested in integrating, but I whipped together a little tutorial on integrating core.async with the shopping form.

@magomimmo
Copy link
Owner

Hi James, sorry to be late. I'll take a look at it this evening and let you know what I think about it.
My best

@magomimmo
Copy link
Owner

Hi James, I still have to take a deeper look at your very nice contribution. I have one question: are we sure that the async stuff is the best solution for managing the updates of the shopping form? Generally speaking async stuff are very useful when you have more things going on at the same time (http://swannodette.github.io/2013/07/12/communicating-sequential-processes/).
That said, probably we should add it after the tutorial on domina events. Have you already a clear idea on where to position the tutorial in the series?

My best

mimmo

@jamesnvc
Copy link
Contributor Author

I don't think that, at this stage, async is necessarily the best way of handling the form, since it is fairly simple. For pedagogical purposes though, I thought it would make sense to start with a fairly simple example that can be made more complicated "as an exercise for the reader". The error highlighting part definitely doesn't require the use of async, but I think that it provides a simple introduction to the use of channels and such. The update-form-only-when-error-free part though, is much more cleanly done using async in my opinion.

I'm not sure as to the position of the entry in the series; I was building off of the last tutorial, but as you say, it would probably fit in best after the tutorial on events.

@Will-Sommers
Copy link

I'm going to work though this(new to CLJS) and would like to post my thoughts here if that is okay.

@magomimmo
Copy link
Owner

Please! No problem.

Sorry to be in late updating modern-cljs….I have a lot of stuff to add….I don't find the time it requires to put them down in words…

cheers,
mimmo

On Jan 14, 2014, at 10:00 PM, Will Sommers [email protected] wrote:

I'm going to work though this(new to CLJS) and would like to post my thoughts here if that is okay.


Reply to this email directly or view it on GitHub.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants