From d0b10696973a5f478b1af7494691b42b7909de1c Mon Sep 17 00:00:00 2001
From: Sarah Funkhouser <147884153+golanglemonade@users.noreply.github.com>
Date: Tue, 28 Jan 2025 15:12:42 -0700
Subject: [PATCH] allow tfa to be disabled without removing the settings (#422)

Signed-off-by: Sarah Funkhouser <147884153+golanglemonade@users.noreply.github.com>
---
 internal/ent/hooks/usersettings.go | 15 ---------------
 1 file changed, 15 deletions(-)

diff --git a/internal/ent/hooks/usersettings.go b/internal/ent/hooks/usersettings.go
index 707ddccb..6c873b57 100644
--- a/internal/ent/hooks/usersettings.go
+++ b/internal/ent/hooks/usersettings.go
@@ -13,7 +13,6 @@ import (
 	"github.com/theopenlane/core/internal/ent/generated"
 	"github.com/theopenlane/core/internal/ent/generated/hook"
 	"github.com/theopenlane/core/internal/ent/generated/privacy"
-	"github.com/theopenlane/core/internal/ent/generated/tfasetting"
 	"github.com/theopenlane/core/internal/ent/generated/user"
 	"github.com/theopenlane/core/internal/ent/generated/usersetting"
 	"github.com/theopenlane/core/internal/ent/privacy/rule"
@@ -29,20 +28,6 @@ func HookUserSetting() ent.Hook {
 				return nil, rout.InvalidField(rout.ErrOrganizationNotFound)
 			}
 
-			// delete tfa setting if tfa is disabled
-			tfaEnabled, ok := m.IsTfaEnabled()
-			if ok && !tfaEnabled {
-				userID, err := auth.GetUserIDFromContext(ctx)
-				if err != nil {
-					return nil, err
-				}
-
-				_, err = m.Client().TFASetting.Delete().Where(tfasetting.OwnerID(userID)).Exec(ctx)
-				if err != nil {
-					return nil, err
-				}
-			}
-
 			return next.Mutate(ctx, m)
 		})
 	}, ent.OpUpdate|ent.OpUpdateOne)