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

Updating DAO->Governance menu to indicate vote possible #6324

Merged
merged 1 commit into from Sep 21, 2022
Merged

Updating DAO->Governance menu to indicate vote possible #6324

merged 1 commit into from Sep 21, 2022

Conversation

ghost
Copy link

@ghost ghost commented Aug 14, 2022

Fixes #3744

Replacing ChangeListener with EasyBind.Subscription causes immediate invoke change at the time of binding creation, which fixes posted issue

@ghost ghost self-requested a review August 15, 2022 09:33
@ghost
Copy link

ghost commented Aug 15, 2022

@xyzmaker123 while testing in Bisq prior to this PR I'm unable to recreate the problem.

  • Start localnet cluster
  • Create proposal to change parameter
  • Advance blocks to blind vote
  • See asymmetrical menus from the proposal creator and the voter

Both menu buttons immediately change to VOTE ON PROPOSALS when the block advances to the blind vote stage.
I'd like to recreate the issue in order to test the fix. Please advise.

Screenshot:
image

@ghost
Copy link
Author

ghost commented Aug 15, 2022

@jmacxx You need to be on another view during advancing blocks to blind vote and then switch to DAO->Governance.

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK

Regtest Testing note: the DAO facade phase property stays as Blind vote for 1 block into the break period. But it seems minor and does not really cause any trouble except the button says vote on proposals for one extra block actually in the break period. Possibly a minor bug in the DAO.

@ripcurlx
Copy link
Contributor

As this is touching the DAO code it is necessary to get a ACK from @ManfredKarrer as well prior merging.

@ghost
Copy link
Author

ghost commented Aug 17, 2022

@ripcurlx It doesn't touch any domain code - only simple switch from ChangeListener to EasyBind.Subscription on UI level.

@ripcurlx
Copy link
Contributor

As this is touching the DAO code it is necessary to get a ACK from @ManfredKarrer as well prior merging.

Yes, I'm aware of that, but we had cases in the past where also UI code caused some issues, so this is just an extra safety net. I'll try to ping @ManfredKarrer for a review.

@ripcurlx ripcurlx added this to the v1.9.5 milestone Aug 19, 2022
@ghost ghost mentioned this pull request Sep 5, 2022
@ripcurlx ripcurlx modified the milestones: v1.9.5, v1.9.6 Sep 9, 2022
Copy link
Contributor

@ManfredKarrer ManfredKarrer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK

Copy link
Contributor

@ripcurlx ripcurlx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK - based on #6324 (review)

@ripcurlx ripcurlx merged commit dc58029 into bisq-network:master Sep 21, 2022
@ghost ghost mentioned this pull request Oct 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DAO->Governance menu not updating to indicate vote possible
2 participants