cowprotocol / cowprotocol/token-lists

No validation on removeToken form

Open
#105 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug good first issue
Dominant language
TypeScript
Stars
16
Forks
35
PR merge metrics
No merged PRs in 30d

Description

There is no validation for fields on the Remove token form:

  1. Instead of the address I can fill in anything I want, even 1 space is enough. It would be great to add validation ad least on an address format
  2. There is no validation for presence of an address on the List (I think, it is nice to have)
  3. There is no validation if an address belongs to a selected network or not (see #102)

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Inspect the Remove token form and the token list handling described in the issue, then review issue #102 for the selected-network validation context. Done means rejecting malformed or missing addresses and detecting addresses that do not belong to the selected network.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
frontend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.