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.

Sign up for GitHub

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

Jump to bottom

Revoke authority. Unexpected removing of authority #1629

Open
stepanLav opened this issue Apr 4, 2024 · 2 comments
Open

Revoke authority. Unexpected removing of authority #1629

stepanLav opened this issue Apr 4, 2024 · 2 comments
Assignees
@Asmadek
Labels
Bug Issues & PRs that addressing a bug

Comments

@stepanLav
Copy link
Member

Description

When user remove delegated authority - it may remove more than one.

Steps to Reproduce

Preconditions:

user has a wallet with a few delegated authority

  1. Go to wallet details
  2. Remove authorities one by one

Expected Behavior

Successfully Remove all authorities

Actual Behavior

When removing, 5 delegated authorities - 2 authorities was removed unexpectadly from the list, but not from the blockchain

Environment

3530845

Screenshots

unexpected_remove.mp4

log:
nova-spektr.log
@stepanLav stepanLav added the Bug Issues & PRs that addressing a bug label Apr 4, 2024
@pgolovkin pgolovkin assigned Asmadek and unassigned pgolovkin Apr 4, 2024
@Asmadek Asmadek moved this from Todo to Development in Nova Spektr development Apr 5, 2024
@Asmadek Asmadek moved this from Development to In Review in Nova Spektr development Apr 5, 2024
@Asmadek Asmadek linked a pull request Apr 5, 2024 that will close this issue
feat: add remove proxy with proxy #1633
Merged
@Asmadek Asmadek moved this from In Review to Ready for Test in Nova Spektr development Apr 8, 2024
@stepanLav stepanLav moved this from Ready for Test to Testing in Nova Spektr development Apr 10, 2024
@stepanLav
Copy link
Member Author

Status verification: ❌

What has been verified:
still has problem with removing
4d1d9b4

remove_current_wallet.mp4

@stepanLav stepanLav reopened this Apr 10, 2024
@github-project-automation github-project-automation bot moved this from Testing to Development in Nova Spektr development Apr 10, 2024
@stepanLav stepanLav moved this from Development to Testing in Nova Spektr development Apr 10, 2024
@Asmadek Asmadek removed a link to a pull request Apr 10, 2024
feat: add remove proxy with proxy #1633
Merged
@Asmadek Asmadek moved this from Testing to Development in Nova Spektr development Apr 10, 2024
@Asmadek Asmadek moved this from Development to Ready for Test in Nova Spektr development Apr 10, 2024
@stepanLav stepanLav moved this from Ready for Test to Testing in Nova Spektr development Apr 11, 2024
@stepanLav
Copy link
Member Author

Status verification: ✅

@stepanLav stepanLav moved this from Testing to Done in Nova Spektr development Apr 11, 2024
@pgolovkin pgolovkin modified the milestone: Milestone 4 Nov 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Asmadek Asmadek

Labels
Bug Issues & PRs that addressing a bug
Projects
Nova Spektr development
Status: Done
Milestone
No milestone
Development

No branches or pull requests
3 participants
@pgolovkin @Asmadek @stepanLav