-
Notifications
You must be signed in to change notification settings - Fork 242
Description
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: FeatureType: Enhancement - Add your estimation
Estimations
| 3 | Mild efort | A day | Low complexity | Low |