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

Remove coin selection from core library #3

Closed
ProofOfKeags opened this issue Jul 13, 2022 · 0 comments · Fixed by #26
Closed

Remove coin selection from core library #3

ProofOfKeags opened this issue Jul 13, 2022 · 0 comments · Fixed by #26
Labels
breaking change This change will require a major version upgrade enhancement New feature or request
Milestone
0.1.0 Release

Comments

@ProofOfKeags
Copy link
Collaborator

Coin selection algorithms are a fuzzy solution, as such they need to be flexible and swappable. Ideally we make it easy to create a means of plugging in your own and also export some simple or common ones.

Suggested by @wraithm.
@ProofOfKeags ProofOfKeags added enhancement New feature or request needs concept ACK Idea for a change, not yet decided if the change is good breaking change This change will require a major version upgrade labels Jul 13, 2022
@ProofOfKeags ProofOfKeags linked a pull request Sep 8, 2022 that will close this issue
Feature/heavy dep purge #12
Closed
@ProofOfKeags ProofOfKeags mentioned this issue Sep 8, 2022
Closed
@ProofOfKeags ProofOfKeags removed the needs concept ACK Idea for a change, not yet decided if the change is good label Sep 30, 2022
@ProofOfKeags ProofOfKeags added this to the 0.1.0 Release milestone Sep 30, 2022
This was referenced Oct 1, 2022
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
breaking change This change will require a major version upgrade enhancement New feature or request
Projects
None yet
Milestone
0.1.0 Release
1 participant
@ProofOfKeags