Code review comment for lp:~dholbach/loco-team-portal/profile

Revision history for this message
Daniel Holbach (dholbach) wrote :

> Or cronjob based.. At the moment, that wouldn't be intensive as there aren't
> that many authenticated users, but as LD gets more popular this could be a
> real bottleneck.

Not only on login, but also when a userprofile is created (admin/owners through lpupdate management command).

> Do we *want* a list of all members, or stick to generating profile on login?

The problem we have right now is that loco contacts can only be chosen from the list of existing profiles (logged in people + lp admins + lp owner).

« Back to merge proposal