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.

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

Refactors wallet handling to allow for adapters. #7

Merged
merged 14 commits into from
Mar 21, 2024

Conversation

jamesduncombe
Copy link
Contributor

@jamesduncombe jamesduncombe commented Jan 30, 2024

This PR:

Housekeeping 🛠️

  • Updates testing matrix to include OTP v25 and Elixir v1.14.5.

Wallet handling

  • Refactors TTEth.Wallet so that it can handle multiple adapters.
  • Adds TTEth.Behaviours.Wallet for wallet adapters:
    • new/1 - Builds needed attributes to initialise the underlying adapter.
    • wallet_attrs/2: Given a wallet adapter and map return public wallet attributes (public key etc).
    • sign/2: Given a wallet adapter and and hash, return a signature.

⚠️ This will make #6 redundant ⚠️

@jamesduncombe jamesduncombe force-pushed the features/refactors-wallet-handling branch from 76d791d to a86fa66 Compare January 30, 2024 14:15
@jamesduncombe jamesduncombe added enhancement New feature or request beta Still very much in testing mode. labels Jan 30, 2024
@jamesduncombe jamesduncombe changed the title wip Refactors wallet handling. Jan 30, 2024
@jamesduncombe jamesduncombe changed the title Refactors wallet handling. Refactors wallet handling to allow for adapters. Jan 30, 2024
@jamesduncombe jamesduncombe force-pushed the features/refactors-wallet-handling branch 2 times, most recently from a770c10 to 1944254 Compare January 31, 2024 11:24
@jamesduncombe jamesduncombe force-pushed the features/refactors-wallet-handling branch 12 times, most recently from e318d11 to e51e52c Compare February 1, 2024 12:41
@jamesduncombe jamesduncombe force-pushed the features/refactors-wallet-handling branch from b078120 to 8b20d35 Compare February 1, 2024 18:01
@jamesduncombe jamesduncombe force-pushed the features/refactors-wallet-handling branch from 8b20d35 to 81e266b Compare February 1, 2024 18:06
@jamesduncombe jamesduncombe force-pushed the features/refactors-wallet-handling branch from 902f87f to bd8e1ee Compare March 11, 2024 12:24
@jamesduncombe jamesduncombe marked this pull request as ready for review March 19, 2024 16:31
@jamesduncombe jamesduncombe removed the beta Still very much in testing mode. label Mar 21, 2024
@jamesduncombe jamesduncombe merged commit 9c53b3a into main Mar 21, 2024
6 checks passed
@jamesduncombe jamesduncombe deleted the features/refactors-wallet-handling branch March 21, 2024 10:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant