Code review comment for lp://staging/~nueces/lalita/support-private-channels

Revision history for this message
Facundo Batista (facundo) wrote :

This looks nice! However, some details:

- point at the end of the docstring

- remove blank in the ".get (...)"

And what it mandates a need fixing here: You need to provide test cases for this new feature.

Thank you very much!!

review: Needs Fixing

« Back to merge proposal