-
Notifications
You must be signed in to change notification settings - Fork 401
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
Refactor Parser docs code snippets #1371
Comments
This was exactly what I was looking for, and worked great. Some comments on making the docs better. In the Parsing event section, the
but this has an implication on the Lambda response that could be more explicitly called out. I mainly noticed this because it caused a change in my unit test behavior. The next section Also, the first example had the order: classes, hander def, payload def, invoke hander - and the next example was: classes, payload, function def, (no invoke example). The first example seemed to flow better, and these could be normalized. |
Hey @hakanson! Hope you are well. Now I'm refactoring all the docs, examples and adding more details in general. I'm planning to send a PR tomorrow or Wednesday.. |
Yes, I have some commitments coming but will do my best. |
|
This is now released under 3.3.0 version! |
No description provided.
The text was updated successfully, but these errors were encountered: