这是indexloc提供的服务,不要输入任何密码
Skip to content

Further Wallet Provider abstraction #975

@rob1997

Description

@rob1997

Summary
We can technically make all RPC provider signers and transaction executors into one.

Suggested features or ways to address the summary
One signer and one transaction executor for all IwalletProviders

Motivation
This will improved reusability, maintainability and readability in our code base with less parts. More over just like how the previous abstraction allowed every transaction executor to poll txns we can now verify every signature.

Acceptance Criteria
Signing and sending txn works like before
reusability, maintainability and readability increased in the codebase
less parts and signature verification

Blockers
(OPTIONAL) [Do you see any blocker that needs to be done before implementing this feature?]

Dependencies
(OPTIONAL) [Do you know about dependencies that directly affect this issue?]

Additional Context
(OPTIONAL) [Add any other context, files, images (screenshots/videos), diagrams, or documents about the feature request here.]

Final Steps

  • Give it one of these labels Type: Feature Type: Enhancement
  • Add your estimation

Estimations

| 3 | Mild efort | A day | Low complexity | Low |

Metadata

Metadata

Assignees

Labels

Type: EnhancementAdded to issues and PRs when a change includes improvements or optimizations.Type: MaintenanceAdded to issues and PRs when a change is for repository maintenance , such as CI or linter changes.

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions