Conversation
Member
This pull request builds on top of #227.
Thanks to @broberts-dev for putting the time into these changes!
I rearranged some commits and added a few improvements to the documentation.
Resolves and Closes #227.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@SimonFrings Thank you for looking into this and taking this over!
Most of the suggestions look good to me, what do you think about the documentation for the callback functions? (see also other callbacks)
clue approved these changes Nov 12, 2021
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@SimonFrings Thanks for the update, changes LGTM! ![]()