fix: repository lookup select contract form

hashgraph / hedera-sourcify

Tools for verifying Hedera smart contracts using standard open source libraries.

Apache License 2.0

6 stars 7 forks source link

fix: repository lookup select contract form #100

Closed acuarica closed 6 months ago

acuarica commented 6 months ago

Description:

Use checksummed address lookup in the repository select-contract-form. This allows the user to use the form with non-checksummed addresses.

Related issue(s):

Fixes #91

Notes for reviewer:

Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

acuarica commented 6 months ago

@Nana-EC @svienot The PR diff looks like a lot has changed, but I only added some lines in App.js and package.json. This is because IntelliJ reordered the files within the patch.

Shall we use a different approach, e.g., git diff to generate the patch consistently?

acuarica commented 6 months ago

Looks good, works well.

@svienot just to double-check, did you manage to verify that this PR works as expected? Applying the patch, building the image and make a lookup in the repository?

If that's the case, I will go ahead and merge this. I wasn't able to verify it myself because of https://github.com/hashgraph/hedera-sourcify/issues/103.

svienot commented 6 months ago

@acuarica yes I did check this. We should be good to go.