Added to EventContext.renderEach documentation #231

Merged
merged 2 commits into from Aug 26, 2014

Conversation

Projects
None yet
2 participants
@rikkiprince
Contributor

rikkiprince commented Aug 17, 2014

It took me quite a lot of experimentation and reading the sammy source code to understand how the different arguments worked, especially as the provided examples plainly ignore the name argument without explanation. I also explained that the data can come from the previous step in the chain (is that the right terminology for previous_content?), as it's used in the sample on the front page, but not explained.

I also fixed a typo.

rikkiprince added some commits Aug 17, 2014

Improved EventContext.renderEach documentation.
Explained use of data and name arguments, and provided a new example to
better demonstrate the subtleties of these arguments.

endor added a commit that referenced this pull request Aug 26, 2014

Merge pull request #231 from rprince/master
Added to EventContext.renderEach documentation

@endor endor merged commit 71a6210 into quirkey:master Aug 26, 2014

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