Fix metric tracking to not blow up when signup fails

app/controllers/authentication/registrations_controller.rb

@@ -4,7 +4,6 @@ class Authentication::RegistrationsController < Devise::RegistrationsController
   before_action :configure_permitted_parameters, if: :devise_controller?
   before_action :set_invite_token, only: [:new, :create]
   before_action :set_invite, only: [:new, :create]
-  after_action :identify_team, only: [:create]
   alias_method :user, :resource
   helper_method :user
 
@@ -40,6 +39,7 @@ def create
     end
 
     finish_sign_up
+    identify_team
   end
 
   protected
@@ -96,10 +96,11 @@ def sign_up_params_for_invites
   end
 
   def sign_up_email
-    sign_up_params[:email]
+    sign_up_params[:email].downcase
   end
 
   def identify_team
-    SiteAnalytics.identify_and_group(user.reload, user.reload.organizations.first)
+    return unless user.persisted?
+    SiteAnalytics.identify_and_group(user, user.organizations.first)
   end
 end

app/models/accounts/invite.rb

@@ -32,6 +32,7 @@ class Accounts::Invite < ApplicationRecord
       message: :invalid_format
     }
 
+  before_save -> { self.email = email.downcase }
   before_save :add_recipient
   before_create :generate_token