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

Aggregate Builder. Address issue. Edit issue #1748

Closed
OlegMakarenko opened this issue Oct 21, 2021 · 1 comment · Fixed by #1754
Closed

Aggregate Builder. Address issue. Edit issue #1748

OlegMakarenko opened this issue Oct 21, 2021 · 1 comment · Fixed by #1754
Assignees
Labels
P1 Issue Breaking issue effecting large number of users, Security issue, Risk to users funds

Comments

@OlegMakarenko
Copy link
Contributor

The address issue

I have two accounts. The first one is Seed Account 1 ...-QDQ. Another one Multisig ...-OHA

  1. Go to the Aggregate page
  2. Go to the Transfer transaction tab
  3. First inner transaction. Select Seed Account 1 ...-QDQ as FROM and input Multisig ...-OHA as TO. Click Save button.
  4. Second inner transaction. Select Multisig ...-OHA as FROM and input Seed Account 1 ...-QDQ as TO. Click Save button.
  5. Click Send aggregate button.
  6. In the transaction details the first transfer has Multisig ...-OHA as a signer but should have Seed Account 1 ...-QDQ

The edit issue

  1. Click on the prepared inner transaction to edit it.
  2. The modal box appears without editing form.

Screen recordings are here: https://fex.net/s/bkfxnrr

@OlegMakarenko OlegMakarenko added the P1 Issue Breaking issue effecting large number of users, Security issue, Risk to users funds label Oct 21, 2021
@cryptoBeliever
Copy link
Contributor

I can confirm bugs with the wrong signer and editing issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 Issue Breaking issue effecting large number of users, Security issue, Risk to users funds
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants