Code review comment for lp:~cerberus/nova/nova_notifications

Revision history for this message
Brian Waldon (bcwaldon) wrote :

> Think I covered the rest. Also made changes based on Rick's suggestion

78, 88, 118: I still feel like 'message' on these lines should be 'payload'. It's confusing to give the notify function a 'message' which is encapsulated as the 'payload' in a new type of 'message' structure.

« Back to merge proposal