Skip to content

Conversation

@Agamya-Samuel
Copy link

@Agamya-Samuel Agamya-Samuel commented Jan 28, 2026

Fix voting statistics display issue #325

  • Add backend check to only return statistics when show_stats is enabled
  • Add statistics display section to VoteYesNo component (Accept/Decline counts)
  • Add statistics display section to VoteRating component (star ratings distribution)
  • Implement statistics fetching on component mount and round prop changes
  • Refresh statistics automatically after each vote submission
  • Add CSS styles for statistics display section

Copy link
Member

@mahmoud mahmoud left a comment

Choose a reason for hiding this comment

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

Looking good! Would you mind including a recording of it in action? Just a screen recording of your local is fine.

@Agamya-Samuel
Copy link
Author

Sure @mahmoud

@Agamya-Samuel
Copy link
Author

Before

  • No statistics of Votes (Yes/No and Star Rating) were visible on left sidebar.
2026-01-28.23-53-30.mp4

After

Yes/No Vote:

2026-01-29.01-55-05.mp4

Rating Vote:

2026-01-29.01-45-21.mp4

Only return voting statistics when show_stats is enabled for the round.
This prevents statistics from being returned when the feature is disabled.
- Add statistics display section showing Accept/Decline counts
- Implement statistics fetching on mount and round prop changes
- Refresh statistics automatically after each vote submission
- Add CSS styles for statistics display
- Add statistics display section showing star ratings distribution (1-5 stars)
- Implement statistics fetching on mount and round prop changes
- Refresh statistics automatically after each vote submission
- Add CSS styles for statistics display
Improve vertical alignment of star icons in the statistics section by using
inline-flex with center alignment for better visual consistency.
@Agamya-Samuel Agamya-Samuel force-pushed the fix/325-voting-statistics-display branch from f2aa510 to 1657280 Compare January 29, 2026 08:10
@Agamya-Samuel
Copy link
Author

Hi @mahmoud the PR is ready to be reviewed. I have attached the requested video recording.

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.

2 participants