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

Improve experience, usability giving proposals minimum votes number #13349

Merged
merged 59 commits into from
Dec 20, 2024

Conversation

ElviaBth
Copy link
Contributor

@ElviaBth ElviaBth commented Sep 4, 2024

🎩 What? Why?

This PR is implementing a solution to improve participants' experience and usability by giving a minimum number of votes to proposals. The notifications with the voting rules will always be visible enough and my support may end up being counted with me being aware.

📌 Related Issues

Testing

  1. Log in as administrator.
  2. Enter the admin panel and navigate to a process with a proposals component with a vote limit per participant and minimum votes per user enabled.
  3. See the process in the browser.
  4. Navigate to proposals on the frontend.
  5. See that there is a banner showing the voting rules.
  6. Click on a proposal and see a notification displaying the remaining minimum votes.
  7. Vote for the minimum number of proposals, and then you'll see a notification indicating that you have correctly given all the minimum votes.
  8. When you have not voted for the minimum number of proposals and you click outside the proposal component, then you'll see a modal notifying you that you have pending votes to give.

📷 Screenshots

image
image
voting rules banner
image
voting rules dropdown
image
image
exit_modal
♥️ Thank you!

Copy link

request-info bot commented Sep 4, 2024

It seems like you did not give us much information about what you are trying to do here. We would appreciate it if you could provide us with more info about this issue/PR!

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This pull request does not contain a valid label. Please add one of the following labels: ['type: feature', 'type: change', 'type: fix', 'type: removal', 'target: developer-experience', 'type: internal']

@probot-autolabeler probot-autolabeler bot added the javascript Pull requests that update Javascript code label Sep 5, 2024
github-actions[bot]
github-actions bot previously approved these changes Dec 18, 2024
github-actions[bot]
github-actions bot previously approved these changes Dec 18, 2024
github-actions[bot]
github-actions bot previously approved these changes Dec 18, 2024
@alecslupu
Copy link
Contributor

alecslupu commented Dec 19, 2024

From Code perspective this is ok now, yet i have some issues regarding the usability, and i would really like to discuss with @decidim/product about the way this implemented.

My issue is that when voting is enabled you get a confirmation whether to navigate away from proposal voting, even though you may just browse the proposals. This is different than the Budget functionality as on the budget this posp-up get's activated just when you actually get to participate to voting.

cc @ElviaBth , @andreslucena , @microstudi , @carolromero , @NilHomedes

@microstudi
Copy link
Contributor

What you say have lots of sense. I think you are right, maybe we should show this modal only if the user has started to vote? Is this behaviour also happening even if the user is not logged in?

@alecslupu
Copy link
Contributor

What you say have lots of sense. I think you are right, maybe we should show this modal only if the user has started to vote? Is this behaviour also happening even if the user is not logged in?

TBH, i have not checked the visitor behavior, but i have discussed a lot with @ElviaBth yesterday, so she knows my pain and doubts.

@ElviaBth could you check and reply here, so that Product team can also see ?

@ElviaBth
Copy link
Contributor Author

What you say have lots of sense. I think you are right, maybe we should show this modal only if the user has started to vote? Is this behaviour also happening even if the user is not logged in?

TBH, i have not checked the visitor behavior, but i have discussed a lot with @ElviaBth yesterday, so she knows my pain and doubts.

@ElviaBth could you check and reply here, so that Product team can also see ?

Yes, is the same behavior for visitor.

@microstudi
Copy link
Contributor

@alecslupu we've discussed this and we will only do this check if there's at least one vote. This will cover the case when non logged users too.

@alecslupu
Copy link
Contributor

@alecslupu we've discussed this and we will only do this check if there's at least one vote. This will cover the case when non logged users too.

Ok, looking forward for a new review.

github-actions[bot]
github-actions bot previously approved these changes Dec 20, 2024
github-actions[bot]
github-actions bot previously approved these changes Dec 20, 2024
@ElviaBth
Copy link
Contributor Author

ElviaBth commented Dec 20, 2024

@alecslupu we've discussed this and we will only do this check if there's at least one vote. This will cover the case when non logged users too.

Ok, looking forward for a new review.

Hi @alecslupu, I have applied the suggested changes to the exit modal behavior and look forward to your review.

Copy link
Contributor

@alecslupu alecslupu left a comment

Choose a reason for hiding this comment

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

👍

Merging with failing spec on Codecov

@alecslupu alecslupu merged commit ea0c1aa into develop Dec 20, 2024
114 of 115 checks passed
@alecslupu alecslupu deleted the feature/proposals_minimum_votes_number branch December 20, 2024 15:30
mllocs pushed a commit that referenced this pull request Jan 9, 2025
…13349)

Co-authored-by: Alexandru Emil Lupu <contact@alecslupu.ro>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contract: 2024-developments Barcelona City Council contract javascript Pull requests that update Javascript code module: core module: proposals type: feature PRs or issues that implement a new feature
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Improve participant experience and usability for giving a minimum number of votes to proposals
5 participants