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

fix: allow undefined froms to pass reconciliation (PR 2) #481

Merged
merged 1 commit into from
Oct 2, 2023

Conversation

kanej
Copy link
Member

@kanej kanej commented Sep 20, 2023

If a from is undefined on the first run, allow it to reconcile on the second run using the account that was selected during initialization of the future on the first run. This is only valid if the first run from account (the default sender on the first run) is still in the list of accounts.

Fixes #413.

@kanej kanej force-pushed the fix/allow-reconciliation-of-undefined-from branch from 9d7fbf0 to 54bb8ef Compare September 20, 2023 16:45
If a `from` is undefined on the first run, allow it to reconcile on the
second run using the account that was selected during initialization of
the future on the first run. This is only valid if the first run from
account (the default sender on the first run) is still in the list of
accounts.

Fixes #413.
@kanej kanej force-pushed the fix/allow-reconciliation-of-undefined-from branch from 54bb8ef to 39bfbb4 Compare September 29, 2023 10:11
@kanej kanej changed the title fix: allow undefined froms to pass reconciliation fix: allow undefined froms to pass reconciliation (PR 2) Sep 29, 2023
@kanej kanej changed the base branch from development to multiple-validation-messages September 29, 2023 10:11
Base automatically changed from multiple-validation-messages to development October 2, 2023 19:27
@zoeyTM zoeyTM merged commit 05725c3 into development Oct 2, 2023
6 checks passed
@zoeyTM zoeyTM deleted the fix/allow-reconciliation-of-undefined-from branch October 2, 2023 19:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Reconciliation of undefined froms is too restrictive
3 participants