Code review comment for lp:~jtv/maas/api-list-networks

Revision history for this message
Raphaƫl Badin (rvb) wrote :

No further remarks :).

[1] and [3] are, I think, blocker but feel free to fix that in follow-up branches.

I see you've added the 'resource_uri' field in https://code.launchpad.net/~jtv/maas/api-add-edit-network/+merge/205539 so it's just a matter of adding a test to fix [3].

review: Approve

« Back to merge proposal