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.
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?