Skip to content

fix drep vote calculation for proposal display tests#4057

Merged
kneerose merged 1 commit into
IntersectMBO:developfrom
JosephRana11:develop
Sep 5, 2025
Merged

fix drep vote calculation for proposal display tests#4057
kneerose merged 1 commit into
IntersectMBO:developfrom
JosephRana11:develop

Conversation

@JosephRana11
Copy link
Copy Markdown
Contributor

List of changes

  • Fixed DRep's Vote calculation logic in Outcome details Page.

Test Cases now Fixed from this change

  • 9G_1. Should display correct vote counts on outcome details page in disconnect state
  • 9G_2. Should display correct vote counts on outcome details page

Checklist

  • related issue
  • My changes generate no new warnings
  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the changelog
  • I have added tests that prove my fix is effective or that my feature works

@kneerose kneerose self-requested a review September 5, 2025 08:39
Copy link
Copy Markdown
Contributor

@kneerose kneerose left a comment

Choose a reason for hiding this comment

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

looks good

@kneerose kneerose merged commit 5ac3838 into IntersectMBO:develop Sep 5, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 5, 2025

@JosephRana11
This PR is in the tag: develop-5ac3838adb4ab41863b17f4b4471009a32b3ec21 , for govtool-metadata-validation service

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 5, 2025

@JosephRana11
This PR is in the tag: develop-5ac3838adb4ab41863b17f4b4471009a32b3ec21 , for govtool-backend service

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 5, 2025

@JosephRana11
This PR is in the tag: develop-5ac3838adb4ab41863b17f4b4471009a32b3ec21 , for govtool-frontend service

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