Skip to content

Remove unused code and images#111

Merged
jeffsmale90 merged 1 commit into
mainfrom
chore/purgeUnusedCode
Jul 16, 2025
Merged

Remove unused code and images#111
jeffsmale90 merged 1 commit into
mainfrom
chore/purgeUnusedCode

Conversation

@jeffsmale90

@jeffsmale90 jeffsmale90 commented Jul 14, 2025

Copy link
Copy Markdown
Contributor

Description

Removes some code and files that are unused.

  • Network and token images are now fetched from MetaMask token image repository, so images and iconConstants.ts are removed
  • RequestHeader is an unnecessary abstraction, so it's pulled back into permissionHandlerContent.tsx
  • isSnapsError was not used, so removed
  • iterator is not used, so was removed
  • updateAccountsOrder and moveToFirstIndex are not used, so removed

Manual testing steps

Should have no impact.

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

@jeffsmale90 jeffsmale90 marked this pull request as ready for review July 15, 2025 01:32
@jeffsmale90 jeffsmale90 requested a review from a team as a code owner July 15, 2025 01:32
@jeffsmale90 jeffsmale90 merged commit ace8d62 into main Jul 16, 2025
16 checks passed
@jeffsmale90 jeffsmale90 deleted the chore/purgeUnusedCode branch July 16, 2025 21:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants