Code review comment for lp:~fgallina/rnr-server/click-reviews-return-useroid-as-username

Revision history for this message
Natalia Bidart (nataliabidart) wrote :

Overall looks good, added a few comments for this implementation.

What I wonder it will not be a better approach to migrate usernames as we go, that means: we update the username value with the openid value, that we could have pre-stored in the instances.

What do you think?

« Back to merge proposal