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

Revision history for this message
Jay Pipes (jaypipes) wrote :

Looks reasonable as a starting point for a watch system. One request, though, could you elaborate in the docstrings what the "model" is expected to be? Mapping? Object? A string?

review: Needs Information

« Back to merge proposal