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

Sometimes co-signor can not be removed #1646

Closed
postoronnii opened this issue Aug 17, 2021 · 4 comments · Fixed by #1595
Closed

Sometimes co-signor can not be removed #1646

postoronnii opened this issue Aug 17, 2021 · 4 comments · Fixed by #1595
Assignees
Labels
P2 Issue Feature or UI issue preventing an action, Incomplete process or crashing during action

Comments

@postoronnii
Copy link

WTR:

  1. Go to https://desktop-build.symboldev.com/
  2. Create multisig account with one co-signor
  3. Try to remove co-signor

Result: Sometimes I am getting the error and co-signor can not be removed:
Uncaught Error: Failure_Multisig_Max_Cosignatories at MapSubscriber.eval (TransactionService.js?6d07:102) at MapSubscriber._next (map.js?ebb6:35) at MapSubscriber.Subscriber.next (Subscriber.js?1453:53) at ThrowIfEmptySubscriber._next (throwIfEmpty.js?b4d5:32) at ThrowIfEmptySubscriber.Subscriber.next (Subscriber.js?1453:53) at TakeSubscriber._next (take.js?b7d7:40) at TakeSubscriber.Subscriber.next (Subscriber.js?1453:53) at MergeMapSubscriber.notifyNext (mergeMap.js?a6c5:85) at SimpleInnerSubscriber._next (innerSubscribe.js?cf86:14) at SimpleInnerSubscriber.Subscriber.next (Subscriber.js?1453:53) hashAndAggregateBonded listener is closed due to error!

@postoronnii postoronnii added the P2 Issue Feature or UI issue preventing an action, Incomplete process or crashing during action label Aug 17, 2021
@postoronnii
Copy link
Author

@bassemmagdy @cryptoBeliever the issue is fixed in this PR #1595

@cryptoBeliever
Copy link
Contributor

@NikolaiB but PR is not accepted yet. Issue should be tested after PR acceptance and next in dev branch after branch. Moving back to code review column.

@cryptoBeliever
Copy link
Contributor

cryptoBeliever commented Aug 20, 2021

@NikolaiB could you retest in dev and close if it's fine?

@postoronnii
Copy link
Author

Tested, the issue is fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P2 Issue Feature or UI issue preventing an action, Incomplete process or crashing during action
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants