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

Commission rates and closed pools #332

Open
mh-concordium opened this issue Jul 6, 2022 · 1 comment
Open

Commission rates and closed pools #332

mh-concordium opened this issue Jul 6, 2022 · 1 comment
Labels
[Prio] Low Should be fixed if time permits but can be postponed. [Type] Bug Something isn't working

Comments

@mh-concordium
Copy link

mh-concordium commented Jul 6, 2022

Bug Description
Attempt to update commission rates on a closed pool will fail, it seems that such a change is not recognized as a change to the pool. I assume that this is handling for the closed pool. However, it seems a bit confusing that we can change rates for pool that is closed but then again those changes are not really enforced.

I assume that priority for this is low.

Steps to Reproduce
Add baker with closed pool ->try to change commission rates.
Update baker -> change commission rates again.

Expected Result
Values should be changed accordingly as selected by user. I assume this because commission rates are presented while signing tx in Ledger app.

Actual Result
Values are not changed in Ledger app. Commission rates stay default values.

Versions

  • Software Version: 1.5.0
  • OS: Windows, Mac
Update.commission.rates.mp4
@mh-concordium mh-concordium added [Prio] Low Should be fixed if time permits but can be postponed. [Type] Bug Something isn't working labels Jul 6, 2022
@mh-concordium mh-concordium changed the title Multi-sig transaction - update commission rates doesn't work Multi-sig transaction - update commission rates Jul 7, 2022
@mh-concordium mh-concordium changed the title Multi-sig transaction - update commission rates Commission rates and closed pools Jul 7, 2022
@shjortConcordium
Copy link
Contributor

shjortConcordium commented Jul 19, 2022

Yeah, it should either skip the commission page, or put the changes into the transaction.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Prio] Low Should be fixed if time permits but can be postponed. [Type] Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants