Skip to content

Commit

Permalink
Remove account list transaction on sentry, since the data seems to be…
Browse files Browse the repository at this point in the history
… okay
  • Loading branch information
tommasini committed Jan 6, 2025
1 parent 5fe6ace commit d966587
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 13 deletions.
8 changes: 0 additions & 8 deletions app/components/UI/WalletAccount/WalletAccount.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -25,9 +25,6 @@ import Logger from '../../../util/Logger';
// Internal dependencies
import styleSheet from './WalletAccount.styles';
import { WalletAccountProps } from './WalletAccount.types';
import { TraceName, TraceOperation, trace } from '../../../util/trace';
import { store } from '../../../store';
import { getTraceTags } from '../../../util/sentry/tags';

// TODO: Replace "any" with type
// eslint-disable-next-line @typescript-eslint/no-explicit-any
Expand Down Expand Up @@ -83,11 +80,6 @@ const WalletAccount = ({ style }: WalletAccountProps, ref: React.Ref<any>) => {
accountName={accountName}
accountAvatarType={accountAvatarType}
onPress={() => {
trace({
name: TraceName.AccountList,
tags: getTraceTags(store.getState()),
op: TraceOperation.AccountList,
});
navigate(
...createAccountSelectorNavDetails({
privacyMode,
Expand Down
8 changes: 4 additions & 4 deletions app/components/Views/AccountSelector/AccountSelector.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -55,9 +55,7 @@ const AccountSelector = ({ route }: AccountSelectorProps) => {
const [screen, setScreen] = useState<AccountSelectorScreens>(
AccountSelectorScreens.AccountSelector,
);
useEffect(() => {
endTrace({ name: TraceName.AccountList });
}, []);

useEffect(() => {
if (reloadAccounts) {
dispatch(setReloadAccounts(false));
Expand Down Expand Up @@ -110,7 +108,9 @@ const AccountSelector = ({ route }: AccountSelectorProps) => {
width={ButtonWidthTypes.Full}
size={ButtonSize.Lg}
onPress={() => setScreen(AccountSelectorScreens.AddAccountActions)}
testID={AccountListBottomSheetSelectorsIDs.ACCOUNT_LIST_ADD_BUTTON_ID}
testID={
AccountListBottomSheetSelectorsIDs.ACCOUNT_LIST_ADD_BUTTON_ID
}
/>
</View>
</Fragment>
Expand Down
1 change: 0 additions & 1 deletion app/util/trace.ts
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,6 @@ export enum TraceName {
SwitchBuiltInNetwork = 'Switch to Built in Network',
SwitchCustomNetwork = 'Switch to Custom Network',
VaultCreation = 'Login Vault Creation',
AccountList = 'Account List',
StoreInit = 'Store Initialization',
}

Expand Down

0 comments on commit d966587

Please sign in to comment.