Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Retire ethers.js #3372

Closed
6 of 8 tasks
robertu7 opened this issue May 25, 2023 · 2 comments · Fixed by #3638 or #3740
Closed
6 of 8 tasks

Retire ethers.js #3372

robertu7 opened this issue May 25, 2023 · 2 comments · Fixed by #3638 or #3740
Assignees
@robertu7 @byhow

Comments

@robertu7
Copy link
Contributor

robertu7 commented May 25, 2023 •
edited

Checklist

  • Replace ethers.js/eth-sig-util with viem
    • connector/blockchain/curation
    • mutation/user/claimLogbook
    • mutation/user/generateSigningMessage
    • types/__test__/walletLogin.test
    • uses of utils.isAddress / recoverPersonalSignature
  • Store case sensitive ETH address instead of lowercased
  • Migrate old eth addresses

Refs
@robertu7 robertu7 mentioned this issue Oct 3, 2023
Open
6 tasks
@byhow
Copy link
Contributor

byhow commented Oct 6, 2023

I can take a look at this issue if it has not been worked on yet, thanks

@byhow byhow mentioned this issue Oct 7, 2023
Merged
@robertu7 robertu7 linked a pull request Oct 7, 2023 that will close this issue
migrate ether.js to viem #3638
Merged
@robertu7 robertu7 assigned gary02 and byhow and unassigned gary02 Oct 7, 2023
@robertu7
Copy link
Contributor Author

robertu7 commented Oct 7, 2023

I can take a look at this issue if it has not been worked on yet, thanks

reassigned to you, thanks!

@robertu7 robertu7 self-assigned this Dec 11, 2023
This was referenced Dec 11, 2023
Closed
Closed
@zeckli zeckli mentioned this issue Dec 14, 2023
Closed
@robertu7 robertu7 linked a pull request Dec 27, 2023 that will close this issue
viem #3740
Merged
@robertu7 robertu7 changed the title Retire ethers.js and store case sensitive ETH address Retire ethers.js Jan 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@robertu7 robertu7

@byhow byhow

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

migrate ether.js to viem byhow/matters-server
viem thematters/matters-server
3 participants
@robertu7 @gary02 @byhow