-
Notifications
You must be signed in to change notification settings - Fork 295
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
[DDW-703] Undelegate a wallet #2636
base: develop
Are you sure you want to change the base?
Conversation
…in smaller component
# Conflicts: # source/renderer/app/components/wallet/settings/WalletSettings.js
# Conflicts: # CHANGELOG.md
This PR is blocked by https://input-output.atlassian.net/browse/ADP-1067 |
Hi @danielmain - this link is not accessible to the public, can you please give us an update on this critical missing feature? Thanks. |
This is the PR behind: cardano-foundation/cardano-wallet#3119 We need to clarify with the cardano-wallet Team how to integrate it now in Daedalus. By the way, I don't personally see this as a critical feature. You can still re-delegate your wallet to a different pool anytime. |
I just learned that delegated ADA can still be transferred!? |
This PR CHANGES.
The undelegation feature is partially completed. It is currently disabled since we needed to cover pending delegation statuses after a wallet is undelegated.
Important to understand: When wallet is undelegated, rewards address is deregistered so rewards cannot be collected. Even though a wallet will be earning rewards for two epochs after it was undelegated, the rewards will go to the Cardano treasury.
Todos
Screenshots
Testing Checklist
Review Checklist
Basics
feature
/bug
/chore
,release-x.x.x
)yarn test
)yarn dev
)yarn package
/ CI builds)yarn flow:test
)yarn lint
)yarn prettier:check
)yarn manage:translations
produces no changes)yarn storybook
)yarn.lock
file is updatedCode Quality
Testing
After Review
done
column on the YouTrack board