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

new features/refactoring #10

Open
wants to merge 53 commits into
base: main
Choose a base branch
from
Open

new features/refactoring #10

wants to merge 53 commits into from

Conversation

rbndg
Copy link
Contributor

@rbndg rbndg commented Nov 20, 2024 •
edited
Loading

Fixed/Refactor:

  • Removed all inline imports
  • Fixed node.js / AI example apps
  • Moved some classes like state and others into their own files
  • ws-bare: This module needed keep alive logic implemented as its not supported by bare's ws module.

New Features:

  • TxEntry Class: The format for returning data from doing a new transaction or fetching tx history from any asset
  • PluginBase Class: Base class that has some methods that allows a plugin's methods to be exposed to the outside.
    • Updated Wallet Pay needed changes for this
  • ConnectionManager: Base Class to be extended by blockchain data providers that handles connection states and reconnection logic
    • The base class Provider has been updated to to use ConnectionManager
  • WalletPay Module Info: _getModuleInfo, retrieves version and name of the asset. This is used by the wallet importing logic in the app to determine comparability of an exported wallet.
  • WalletManager: A module for applications that will need multiple instances of lib-wallet. Handles logic of loading wallet, importing. This is used by lib-wallet-bare.
  • New tests for some of these new features

rbndg and others added 30 commits November 15, 2024 19:29
@rbndg
exportWallet
18d946c
@rbndg
export/import wallet
742cfb9
@ceeb4e
feat: add generic wallet pay class
87bec99 
- Based on wallet-pay-eth
- Provide generic logic for wallets using similar implementation as eth
- Add generic hdwallet, provider and state
@rbndg
Merge branch 'tetherto:main' into export-wallet
c79d217
@rbndg
use alternative secp
8b00a0e
@rbndg
use alternative secp
656d0fd
@rbndg
use alternative secp
623f46a
@rbndg
removed wallet-lib.js example
d88f92f
@rbndg
removed wallet-lib.js example, fixed tests
9ca4b5b
@rbndg
removed wallet-lib.js example, fixed tests
2e60755
@rbndg
ws-bare fix
9d3bc96
@rbndg
enable wallet import and fix ws-bare
d4a25e0
@rbndg
moved wallet-import
8d0d6be
@rbndg
moved wallet-import
4ef92e7
@rbndg
multi wallet manager
e737a7a
@rbndg
multi wallet manager import
81b5fb5
@rbndg
refactored export/import
b976a3a
@ceeb4e
chore: linter
fb9b55b
@ceeb4e
chore: linters
ee77fcb
@ceeb4e
chore: linters
e0c9847
@rbndg
WIP
49278d4
@rbndg
WIP: wallet manager
bda4d7f
@rbndg
WIP: wallet manager
65db79f
@rbndg
WIP: wallet manager
a47c3db
@rbndg
WIP: wallet manager
63629c3
@ceeb4e
Merge branch 'main' into generic-wallet-pay
4838908
@rbndg
Merge remote-tracking branch 'ceeb/generic-wallet-pay' into export-wa…
ffab51a 
…llet
@rbndg
WIP: TxEntry class
9796465
@rbndg
WIP
5570a8a
@rbndg
wip:connection manager
22fa7b9
@rbndg rbndg changed the title Feat: import and export wallet new features: export wallet, TxEntry, Plugin system Jan 7, 2025
rbndg
rbndg commented Jan 7, 2025
View reviewed changes
src/lib/wallet-pay-generic.js
@@ -0,0 +1,328 @@
'use strict'
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file is unchanged from one in master

@rbndg rbndg changed the title new features: export wallet, TxEntry, Plugin system new features/refactoring Jan 7, 2025
@rbndg rbndg mentioned this pull request Jan 7, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rbndg @ceeb4e