fuels-wallet icon indicating copy to clipboard operation
fuels-wallet copied to clipboard

Receive screen address not checksummed

Open LuizAsFight opened this issue 1 year ago • 5 comments

Image

LuizAsFight avatar Dec 16 '24 17:12 LuizAsFight

I am Super Franky, a software developer with 3+ years of experience, and I have worked with frontend technologies like React, NextJS, JavaScript/TypeScript. I am also proficient in Cairo for smart contract development. I'm a recent graduate of Web3bridge Africa, the foremost and premier blockchain developer community. I have made contributions to many open-source projects, and I look forward to contributing to this one. My approach to this will be to:

  1. Review the current logic that handles Ethereum addresses to understand how they are being received and displayed, especially when the address is not checksummed.

  2. Implement a function to checksum the received address before displaying it to the user. This will involve converting the address to lowercase, applying the Keccak-256 hash to determine the appropriate capitalization, and then displaying the checksummed address.

  3. Ensure compatibility with existing components and make sure that addresses are shown in the checksummed format, whether they are manually entered or received from external sources.

  4. Write unit tests to verify that the address is correctly checksummed, both when received from an external source and when displayed in the UI. Also, check that no issues arise when displaying the address.

ET: 3 days

FrankiePower avatar Dec 17 '24 12:12 FrankiePower

I'd love to work on this!

danielrobotz avatar Dec 19 '24 11:12 danielrobotz

I'd love to work on this!

Sjoseph432 avatar Dec 20 '24 13:12 Sjoseph432

May I take care of this?

Nityam573 avatar Dec 21 '24 06:12 Nityam573

These comments are from ODHack participants. The issues are being handled and assigned through the ODHack platform. While we appreciate the interest, each issue can only be assigned to one person. We apologize if you were not able to get an assignment.

LuizAsFight avatar Jan 20 '25 18:01 LuizAsFight