Skip to content

Conversation

@AntonioVentilii
Copy link
Collaborator

Motivation

User tokens are deprecated, so we can easily remove them from the flow to load ERC20 tokens.

@AntonioVentilii AntonioVentilii marked this pull request as ready for review January 8, 2026 09:50
Copilot AI review requested due to automatic review settings January 8, 2026 09:50
@AntonioVentilii AntonioVentilii requested a review from a team as a code owner January 8, 2026 09:50
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request removes the deprecated ERC20 user tokens loading functionality from the frontend codebase. User tokens are being phased out in favor of custom tokens.

Key changes:

  • Removed loadErc20UserTokens function and all related user token loading logic
  • Removed over 200 lines of tests for user token loading functionality
  • Updated loadErc20Tokens to only load default and custom tokens
  • Cleaned up deprecated type definitions in user-token.ts

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
src/frontend/src/eth/services/erc20.services.ts Removed user token loading functions, updated imports, and simplified the main loadErc20Tokens function to only load default and custom tokens
src/frontend/src/lib/types/user-token.ts Removed deprecated LoadUserTokenParams type definition
src/frontend/src/tests/eth/services/erc20.services.spec.ts Removed comprehensive test suite for loadErc20UserTokens functionality (198 lines)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@AntonioVentilii AntonioVentilii added this pull request to the merge queue Jan 8, 2026
Merged via the queue into main with commit 7b06a73 Jan 8, 2026
78 checks passed
@AntonioVentilii AntonioVentilii deleted the feat-frontend/Remove-loading-of-ERC20-user-tokens branch January 8, 2026 12:34
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