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.

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

[CHIA-1067] Integrate coin selection with action scopes #18424

Merged
merged 12 commits into from
Aug 19, 2024

Conversation

Quexington
Copy link
Contributor

@Quexington Quexington commented Aug 7, 2024

Right now, each wallet individually selects coins based on the current state of the DB. This means that if you’re attempting to generate multiple transactions that depend on each other at a time, you have to manually edit the action scope’s TX config to indicate what coins are selected.

To avoid this, we move this functionality into the action scope so that selected coins are visible to all future transaction generators. When selected coins are added here, we ignore them when we go to select coins again.

Base automatically changed from quex.move_tx_config_into_action_scopes to main August 7, 2024 18:27
@github-actions github-actions bot added merge_conflict Branch has conflicts that prevent merge to main labels Aug 7, 2024
@Chia-Network Chia-Network deleted a comment from github-actions bot Aug 7, 2024
@Chia-Network Chia-Network deleted a comment from github-actions bot Aug 7, 2024
@Chia-Network Chia-Network deleted a comment from socket-security bot Aug 7, 2024
@Quexington Quexington added Changed Required label for PR that categorizes merge commit message as "Changed" for changelog Exclude_Notes Use this label if the changes in the PR should be excluded from the release notes labels Aug 7, 2024
@Quexington Quexington changed the base branch from main to quex.wallet_side_effects_streamable August 7, 2024 19:03
@github-actions github-actions bot removed the merge_conflict Branch has conflicts that prevent merge to main label Aug 7, 2024
@Chia-Network Chia-Network deleted a comment from github-actions bot Aug 7, 2024
Base automatically changed from quex.wallet_side_effects_streamable to main August 14, 2024 22:05
@github-actions github-actions bot added the merge_conflict Branch has conflicts that prevent merge to main label Aug 14, 2024
@Chia-Network Chia-Network deleted a comment from github-actions bot Aug 15, 2024
@github-actions github-actions bot removed the merge_conflict Branch has conflicts that prevent merge to main label Aug 15, 2024
@Chia-Network Chia-Network deleted a comment from github-actions bot Aug 15, 2024
@Quexington Quexington marked this pull request as ready for review August 15, 2024 15:35
@Quexington Quexington requested a review from a team as a code owner August 15, 2024 15:35
@Quexington Quexington added the ready_to_merge Submitter and reviewers think this is ready label Aug 19, 2024
@Starttoaster Starttoaster merged commit 64003a6 into main Aug 19, 2024
374 of 375 checks passed
@Starttoaster Starttoaster deleted the quex.action_scope_selected_coins branch August 19, 2024 16:02
@altendky altendky mentioned this pull request Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changed Required label for PR that categorizes merge commit message as "Changed" for changelog Exclude_Notes Use this label if the changes in the PR should be excluded from the release notes ready_to_merge Submitter and reviewers think this is ready
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants