-
Notifications
You must be signed in to change notification settings - Fork 1
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
Transaction Group needs an atomic option #1503
Comments
The option itself should read 'Require all signed', or similar. |
Discussion needed. Should this 'require all signed' be the default? Then let the creator cancel transactions as needed (if they aren't signed). |
After discussing the issue with @SimiHunjan, the plan will be to add the atomic check box. The default value will be unchecked. Sequential groups are considered atomic, as well. It will therefor check the 'atomic' box, and disable it thus tying the state of atomic and sequential when sequential is selected. |
If a group is atomic, or sequential (which is also atomic), then the transaction group should likely appear in 'Ready to Execute' as a group, instead of individual transactions. This should be discussed further, for transactions groups that are atomic: Transactions stay in a Transaction Group (appearing like they do in 'Ready to Sign') |
Transaction group needs an option to make the group atomic. This would require that all transactions are submitted together, or none will be submitted. This will not ensure that all transactions will reach consensus. It is a pre-check only.
The check would require that all transactions that are not canceled have all the proper signatures before any of them are submitted.
The text was updated successfully, but these errors were encountered: