feat: bring boilerplate Wallet
class in core
library
#1276
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
A lot of projects that use
wallet-selector
are forced to put in their project boilerplate code ofWallet
class over and over againhttps://github.com/phoenixpulsar/auto-chain-verify/blob/master/src/wallets/near.js
https://github.com/aarav1656/near-ai-agent/blob/master/src/app/utils/near-wallet.js
https://github.com/finowl-near/finowl-app/blob/main/frontend/src/app/Wallets/near.js
https://github.com/partagexyz/fans-club/blob/main/src/wallets/near.js
https://github.com/NEARBuilders/devhub-rebuild-with-requests/blob/main/src/wallets/near-wallet.ts
https://github.com/LewinJun/near-demo/blob/0bc0e73ede2175a9b8ed964f66848281d2bb0ca2/my-books/src/wallets/near.js
and many more
This PR addresses the problem by implementing
Wallet
class as part ofwallet-selector-core
library. The class is written to be flexible enough for different use cases.