kpvdr commented on a change in pull request #186: PROTON-2086: Changed API documentation from epydoc to Sphinx URL: https://github.com/apache/qpid-proton/pull/186#discussion_r318608359
########## File path: python/docs/proton.handlers.rst ########## @@ -0,0 +1,106 @@ +########################## +Module ``proton.handlers`` +########################## + +.. currentmodule:: proton.handlers + +Module Summary +############## + +| + +.. autosummary:: + + MessagingHandler + EndpointStateHandler Review comment: I'll remove the following handlers: - EndpointStateHandler - IncomingMessageHandler - OutgoingMessageHandler which will leave only the following: - MessagingHandler - TransactionHandler - TransactionalClientHandler I don't know the transaction handlers well, but looking at the code, it appears that TransactionalClientHandler inherits from both TransactionHandler and MessagingHandler, so is intended to be the one to use for transactions from a client API pov. I think we should also drop TransactionHandler if you agree. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
