Code review comment for lp:~nataliabidart/canonical-identity-provider/user-person-in-team

Revision history for this message
Daniel Manrique (roadmr) wrote :

Looks OK to me, I'd say to put a FIXME or TODO as part of the "remove the ModelBackend later" comment so it's easier to find, but I don't think it's a big deal since this should be resolved before we forget about that :)

review: Approve

« Back to merge proposal