fix: remove hello method (#146)

Consensys · Sep 22, 2023 · 241c1a7 · 241c1a7
1 parent 148e96e
commit 241c1a7
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 38 deletions.
diff --git a/packages/starknet-snap/index.html b/packages/starknet-snap/index.html
@@ -28,7 +28,6 @@ <h1>Hello, Snaps!</h1>
   <br />
 
   <button class="connect">Connect</button>
-  <button class="sendHello">Send Hello</button>
   <button class="sendPing">Send Ping</button>
   <br>
   <label for="targetChainId">Target Chain Id in Hex (optional, default Goerli testnet)</label>
@@ -275,7 +274,6 @@ <h1>Hello, Snaps!</h1>
 
   const snapId = `local:${window.location.href}`;
   const connectButton = document.querySelector('button.connect')
-  const sendHelloButton = document.querySelector('button.sendHello')
   const sendPingButton = document.querySelector('button.sendPing')
   const getErc20BalanceForm = document.getElementById('getErc20Balance')
   const getEstimateFeeForm = document.getElementById('getEstimateFee')
@@ -299,7 +297,6 @@ <h1>Hello, Snaps!</h1>
   const recoverUserAccountsForm = document.getElementById('recoverUserAccounts')
 
   connectButton.addEventListener('click', connect)
-  sendHelloButton.addEventListener('click', sendHello)
   sendPingButton.addEventListener('click', sendPing)
   getDeployedAccContractsForm.addEventListener('submit', getDeployedAccContracts)
   getErc20BalanceForm.addEventListener('submit', getErc20Balance)
@@ -339,24 +336,6 @@ <h1>Hello, Snaps!</h1>
     })
   }
 
-  // here we call the snap's "hello" method
-  async function sendHello() {
-    try {
-      const response = await ethereum.request({
-        method: 'wallet_invokeSnap',
-        params: {
-          snapId, 
-          request: {
-            method: 'hello'
-          }
-        }
-      })
-    } catch (err) {
-      console.error(err)
-      alert('Problem happened: ' + err.message || err)
-    }
-  }
-
   // here we call the snap's "ping" method
   async function sendPing() {
     try {

diff --git a/packages/starknet-snap/src/index.ts b/packages/starknet-snap/src/index.ts
@@ -31,8 +31,6 @@ import { Mutex } from 'async-mutex';
 import { OnRpcRequestHandler } from '@metamask/snaps-types';
 import { ApiParams, ApiRequestParams } from './types/snapApi';
 import { estimateAccDeployFee } from './estimateAccountDeployFee';
-import { DialogType } from '@metamask/rpc-methods';
-import { copyable, heading, panel } from '@metamask/snaps-ui';
 import { logger } from './utils/logger';
 
 declare const snap;
@@ -98,21 +96,6 @@ export const onRpcRequest: OnRpcRequestHandler = async ({ origin, request }) =>
   };
 
   switch (request.method) {
-    case 'hello':
-      logger.log(`Snap State:\n${toJson(state, 2)}`);
-      return snap.request({
-        method: 'snap_dialog',
-        params: {
-          type: DialogType.Alert,
-          content: panel([
-            heading(`${origin}, Hello!`),
-            copyable(
-              `# of accounts: ${state.accContracts.length}\n\n# of netwoks: ${state.networks.length}\n\n# of erc20Tokens: ${state.erc20Tokens.length}\n\n# of txns: ${state.transactions.length}`,
-            ),
-          ]),
-        },
-      });
-
     case 'starkNet_createAccount':
       apiParams.keyDeriver = await getAddressKeyDeriver(snap);
       return createAccount(apiParams);