remove unnecessary types and methods

MetaMask · Sep 15, 2023 · 2c5dcf5 · 2c5dcf5
1 parent 65dd622
commit 2c5dcf5
Showing 1 changed file with 3 additions and 26 deletions.
diff --git a/packages/rpc-methods/src/restricted/manageAccounts.ts b/packages/rpc-methods/src/restricted/manageAccounts.ts
@@ -33,11 +33,6 @@ type Message = Infer<typeof SnapMessageStruct>;
 
 export const methodName = 'snap_manageAccounts';
 
-type AccountConfirmationResult = {
-  confirmed: boolean;
-  accountName?: string;
-};
-
 export type ManageAccountsMethodHooks = {
   /**
    * Gets the snap keyring implementation.
@@ -48,19 +43,9 @@ export type ManageAccountsMethodHooks = {
       message: Message,
     ) => Promise<Json>;
   }>;
-
-  showSnapAccountConfirmation: (
-    origin: any,
-    type: any,
-    content: any,
-    placeholder: any,
-  ) => Promise<AccountConfirmationResult>;
   startApprovalFlow: (opts?: StartFlowOptions) => ApprovalFlowStartResult;
-  requestUserApproval: (
-    opts: AddApprovalOptions,
-  ) => Promise<AccountConfirmationResult>;
+  requestUserApproval: (opts: AddApprovalOptions) => Promise<boolean>;
   endApprovalFlow: ({ id }: EndFlowOptions) => Promise<void>;
-  // setApprovalFlowLoadingText: () => Promise<void>;
   showApprovalSuccess: (opts?: SuccessOptions) => Promise<SuccessResult>;
   showApprovalError: (opts?: ErrorOptions) => Promise<ErrorResult>;
 };
@@ -142,20 +127,13 @@ export function manageAccountsImplementation({
       type: 'snap_manageAccounts:confirmation',
     });
 
-    // eslint-disable-next-line no-console
-    console.log(
-      'SNAPS/ manageAccountsImplementation/ confirmationResult',
-      confirmationResult,
-    );
-    if (confirmationResult.confirmed) {
+    if (confirmationResult) {
       try {
         const account = await keyring.handleKeyringSnapMessage(origin, params);
 
-        const accountName = confirmationResult.accountName ?? '[Empty]';
-
         await showApprovalSuccess({
           flowToEnd: addAccountApprovalId,
-          message: `Added Account: **${accountName}**`,
+          message: 'Your account is ready!',
         });
 
         return account;
@@ -173,7 +151,6 @@ export const manageAccountsBuilder = Object.freeze({
   specificationBuilder,
   methodHooks: {
     getSnapKeyring: true,
-    showSnapAccountConfirmation: true,
     startApprovalFlow: true,
     requestUserApproval: true,
     endApprovalFlow: true,